lib/messages.c: Make message searches stream as well.
authorCarl Worth <cworth@cworth.org>
Tue, 24 Nov 2009 05:47:24 +0000 (21:47 -0800)
committerCarl Worth <cworth@cworth.org>
Tue, 24 Nov 2009 19:33:09 +0000 (11:33 -0800)
commit70962fabf9c57cda5af26c28894fc9371fd085f2
tree4383d87dd3671eddcac1b06b65776232763452ca
parent94eb9aacd40a8aa133f64bbe5ed71c0138fb5ed9
lib/messages.c: Make message searches stream as well.

Xapian provides an interator-based interface to all search results.
So it was natural to make notmuch_messages_t be iterator-based as
well. Which we did originally.

But we ran into a problem when we added two APIs, (_get_replies and
_get_toplevel_messages), that want to return a messages iterator
that's *not* based on a Xapian search result. My original compromise
was to use notmuch_message_list_t as the basis for all returned
messages iterators in the public interface.

This had the problem of introducing extra latency at the beginning
of a search for messages, (the call would block while iterating over
all results from Xapian, converting to a message list).

In this commit, we remove that initial conversion and instead provide
two alternate implementations of notmuch_messages_t (one on top of a
Xapian iterator and one on top of a message list).

With this change, I tested a "notmuch search" returning *many* results
as previously taking about 7 seconds before results started appearing,
and now taking only 2 seconds.
lib/messages.c
lib/notmuch-private.h
lib/query.cc