Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification

Subject: Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification

Date: Fri, 07 Aug 2015 10:16:19 +0200

To: David Bremner, notmuch@notmuchmail.org

Cc:

From: Daniel Schoepe

On Fri, 05 Jun 2015 19:28 +0200, David Bremner wrote:
> +The **lastmod:** prefix can be used to restrict the result by the
> +database revision number of when messages were last modified (tags
> +were added/removed or filenames changed).  This is usually used in
> +conjunction with the **--db-revision** argument to **notmuch search**
> +to find messages that have changed since an earlier query.

It might be helpful to describe why --db-revision is needed here. At
first glance, just using `notmuch search lastmod:UUID..' looks good
enough to get all changed messages since UUID. Indeed, it's not entirely
clear to me under what conditions just using this query causes problems.
Moreover, it would be quite nice and intuitive if that query did the job
without having to constrain the database revision manually as well.

Best regards,
Daniel

• Previous message (by thread): Re: [PATCH 2/6] lib: Add per-message last modification tracking

Thread:

• David Bremner—Revision tracking, round 2 [inbox, unread]
  • David Bremner—[WIP2 01/12] lib: Only sync modified message documents [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 02/12] lib: Add per-message last modification tracking [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 03/12] lib: API to retrieve database revision and UUID [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 04/12] cli: add type introspection to sprinter type [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 05/12] cli: add two sprinter utility functions [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 06/12] cli/show: add extra element to structured output for metadata [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 07/12] lib: Add "lastmod:" queries for filtering by last modification [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 08/12] cli/show: add lastmod to structured output [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 09/12] cli/search: add metadata element to structured output [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 10/12] emacs: convert notmuch-search to format-version 3 [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 11/12] emacs: convert notmuch-tree to format-version 3 [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • David Bremner—[WIP2 12/12] cli: add global option "--db-revision" [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • Gaute Hope—Revision tracking, round 2 [inbox, unread]
    • David Bremner—revision tracking patches, round 1 [inbox, unread]
      • David Bremner—[PATCH 1/6] lib: Only sync modified message documents [inbox, notmuch::patch, notmuch::pushed, unread]
        • David Bremner—Re: [PATCH 1/6] lib: Only sync modified message documents [inbox, unread]
      • David Bremner—[PATCH 2/6] lib: Add per-message last modification tracking [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Daniel Schoepe—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
          • David Bremner—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
            • Austin Clements—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
          • David Bremner—[PATCH] fixup! lib: Add per-message last modification tracking [inbox, unread]
      • David Bremner—[PATCH 3/6] lib: API to retrieve database revision and UUID [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Tomi Ollila—Re: [PATCH 3/6] lib: API to retrieve database revision and UUID [inbox, unread]
      • David Bremner—[PATCH 4/6] cli/count: add --output=modifications [inbox, notmuch::obsolete, notmuch::patch, unread]
      • David Bremner—[PATCH 5/6] cli: add global option "--db-revision" [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Daniel Schoepe—Re: [PATCH 5/6] cli: add global option "--db-revision" [inbox, unread]
          • David Bremner—Re: [PATCH 5/6] cli: add global option "--db-revision" [inbox, unread]
      • David Bremner—[PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Daniel Schoepe—Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, unread]
          • David Bremner—Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, unread]