notmuch/lib/message.cc, branch 0.32thread-based email index, search, and tagginghttps://git.notmuchmail.org/git/notmuch/atom?h=0.322021-04-18T12:50:36Zlib: directly traverse postlists in _n_message_delete2021-04-18T12:50:36ZDavid Bremnerdavid@tethera.net2021-04-16T12:00:39Zurn:sha1:9ad19e4454ff98342a6c0ec5a7e44c8c516e6fcd
This is intended to fix the slow behaviour of "notmuch new" (and possibly
"notmuch reindex") when large numbers of files are deleted.
The underlying issue [1] seems to be the Xapian glass backend spending
a large amount of time in db.has_positions when running queries with
large-ish amounts of unflushed changes.
This commit removes two uses of Xapian queries [2], and replaces them with
an approximation of what Xapian would do after optimizing the
queries. This avoids the calls to has_positions (which are in any case
un-needed because we are only using boolean terms here).
[1] Thanks to "andres" on IRC for narrowing down the performance
bottleneck.
[2] Thanks to Olly Betts of Xapian fame for talking me a through a fix
that does not require people to update Xapian.
lib: support splitting mail from database location.2021-03-20T10:39:12ZDavid Bremnerdavid@tethera.net2020-12-31T22:20:31Zurn:sha1:e823d05ae6dc920d4fc9abf774c3d2575d891d7b
Introduce a new configuration value for the mail root, and use it to
locate mail messages in preference to the database.path (which
previously implied the mail messages were also in this location.
Initially only a subset of the CLI is tested in a split
configuration. Further changes will be needed for the remainder of the
CLI to work in split configurations.
lib: publish API for notmuch_database_reopen2021-03-18T11:03:36ZDavid Bremnerdavid@tethera.net2021-03-14T14:01:57Zurn:sha1:1121299905e2b2684d7cc56ec35c26c3a012783e
Include the (currently unused) mode argument which will specify which
mode to re-open the database in. Functionality and docs to be
finalized in a followup commit.
lib: run uncrustify2021-03-13T12:45:34Zuncrustifydavid@tethera.net2021-03-13T12:45:34Zurn:sha1:8aeba1228ace947c1b689ae6ae08db5d53755917
This is the result of running
$ uncrustify --replace --config ../devel/uncrustify.cfg *.c *.h *.cc
in the lib directory
lib: replace use of static_cast for writable databases2020-07-28T11:47:58ZDavid Bremnerdavid@tethera.net2020-07-26T23:31:36Zurn:sha1:a09293793f43b93b7008dd361b192199ad528fca
static_cast is a bit tricky to understand and error prone, so add a
second pointer to (potentially the same) Xapian database object that
we know has the right subclass.
lib: encapsulate the use of notmuch_database_t field 'mode'2020-07-28T11:47:58ZDavid Bremnerdavid@tethera.net2020-07-26T23:31:35Zurn:sha1:d7d4c729ab3b74eaaebe64cb8c7383b0ea06cec0
The plan is to change the underlying representation.
lib: fix exception messages for n_m_message_*2020-07-22T22:52:55ZDavid Bremnerdavid@tethera.net2020-07-22T11:19:28Zurn:sha1:e9867b818b75216427c7190f67354959dfb914db
The original generic handler had an extra '%s' in the format
string. Update tests that failed to catch this because the template to
print status strings checked 'stat', which was not set.
lib: fix return value for n_m_reindex2020-07-20T11:54:42ZDavid Bremnerdavid@tethera.net2020-07-09T00:17:09Zurn:sha1:765ca7bc08f83a3c0f9ebffe58dab03634e45f37
Also update the documentation for the behaviour of n_m_get_thread_id
that this fix relies on.
lib: handle xapian exception in n_m_remove_all_tags2020-07-20T11:54:42ZDavid Bremnerdavid@tethera.net2020-07-09T00:17:04Zurn:sha1:a2b90dc084c0224e56c1e63773048545788c7ad4
At least the exception we already catch should be reported properly.
lib: add notmuch_message_has_maildir_flag_st2020-07-20T11:54:42ZDavid Bremnerdavid@tethera.net2020-07-09T00:17:01Zurn:sha1:b7572ceb143773a988a4852ac3b7c85254009d0e
Initially the new function is mainly tested indirectly via the
wrapper.