Re: [PATCH] lib/notmuch_database_reopen: reload some database metada

Subject:Re: [PATCH] lib/notmuch_database_reopen: reload some database metada

Date:Wed, 12 May 2021 09:07:33 -0300

To:notmuch@notmuchmail.org

Cc:

From:David Bremner


David Bremner <david@tethera.net> writes:

> David Bremner <david@tethera.net> writes:
>
>> In some uses of reopen, new documents and threads maybe have been
>> added, and e.g. compaction may have changed the uuid.
>
> It's quite possible there is more cached data that should be discarded,
> but I'll have to think about that a bit more.
>

The following are at least potential problems, although I think none of
them are as urgent as the doc_id / thread_id issue.

exception_reported    should probably be reset to false

atomic_nesting, atomic_dirty.   Re-opening while in an open atomic
                                section is currently undefined
                                behaviour, should be checked.

features  Theoretically if the user runs notmuch-new in a pre-new hook,
          the database could be upgraded. For now, I suggest not doing
          that. This should be fixabable, but I prefer to take more time.
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: