cf/aa06868a913fe66a5700fe85524ebb7f2c1177

   1 Return-Path: <david@tethera.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5  by arlo.cworth.org (Postfix) with ESMTP id 97C9E6DE00DB\r
   6  for <notmuch@notmuchmail.org>; Fri,  3 Jun 2016 05:55:07 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.012\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.012 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.001, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01]\r
  13  autolearn=disabled\r
  14 Received: from arlo.cworth.org ([127.0.0.1])\r
  15  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16  with ESMTP id RCqoBAeEX32t for <notmuch@notmuchmail.org>;\r
  17  Fri,  3 Jun 2016 05:54:59 -0700 (PDT)\r
  18 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id 8FA876DE02A6\r
  20  for <notmuch@notmuchmail.org>; Fri,  3 Jun 2016 05:54:13 -0700 (PDT)\r
  21 Received: from remotemail by fethera.tethera.net with local (Exim 4.84)\r
  22  (envelope-from <david@tethera.net>)\r
  23  id 1b8obu-0004BK-S1; Fri, 03 Jun 2016 08:53:58 -0400\r
  24 Received: (nullmailer pid 4516 invoked by uid 1000);\r
  25  Fri, 03 Jun 2016 12:54:08 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>, notmuch@notmuchmail.org\r
  28 Subject: Re: [RFC2 Patch 5/5] lib: iterator API for message properties\r
  29 In-Reply-To: <87inxrqyv1.fsf@alice.fifthhorseman.net>\r
  30 References: <1463927339-5441-1-git-send-email-david@tethera.net>\r
  31  <1464608999-14774-1-git-send-email-david@tethera.net>\r
  32  <1464608999-14774-6-git-send-email-david@tethera.net>\r
  33  <8760tthfuy.fsf@zancas.localnet> <87pos1u14p.fsf@alice.fifthhorseman.net>\r
  34  <87eg8ht2sb.fsf@alice.fifthhorseman.net> <87lh2ofpxk.fsf@zancas.localnet>\r
  35  <87inxrqyv1.fsf@alice.fifthhorseman.net>\r
  36 User-Agent: Notmuch/0.22+28~gb9bf3f4 (http://notmuchmail.org) Emacs/24.5.1\r
  37  (x86_64-pc-linux-gnu)\r
  38 Date: Fri, 03 Jun 2016 09:54:00 -0300\r
  39 Message-ID: <8737oufn6f.fsf@zancas.localnet>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42  micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.20\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47  <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Fri, 03 Jun 2016 12:55:07 -0000\r
  56 \r
  57 --=-=-=\r
  58 Content-Type: text/plain\r
  59 \r
  60 Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
  61 \r
  62 >\r
  63 > I think this makes sense, and makes me more comfortable with the overall\r
  64 > idea of this patch series.  maybe it'd be useful to clearly document the\r
  65 > intended scope?\r
  66 >\r
  67 \r
  68 Sure, where do you think that kind of documentation is appropriate?\r
  69 There is the giant comment about the database schema in\r
  70 lib/database.cc. Actually I just noticed I already failed to update that\r
  71 for libconfig stuff.\r
  72 >\r
  73 > From elsewhere:\r
  74 >\r
  75 >  * for messages which have multiple files, which file is actually indexed\r
  76 \r
  77 yes. Although rather than storing that, I think the right answer is more\r
  78 like "all of them".\r
  79 \r
  80 >  * thread-id\r
  81 >  * tag\r
  82 >\r
  83 > we're now talking about adding properties, which are in the "elsewhere"\r
  84 > category, right?\r
  85 \r
  86 Correct.\r
  87 \r
  88 >\r
  89 > It's worth noticing that the stuff in "elsewhere" is the stuff that\r
  90 > won't propagate across a dump/restore unless it's explicitly in the dump\r
  91 > somehow.   We currently fail to restore thread-id and which file is\r
  92 > actually indexed across a dump/restore :/\r
  93 \r
  94 The thread-id is in some sense derived from the message itself. Not in a\r
  95 reproducable way, but still, the dump file is the minimal set of extra\r
  96 data needed to reconstruct an equivalent database (pax threading bugs).\r
  97 \r
  98 > I think you've convinced me that it's good to go ahead with the\r
  99 > properties, assuming it's scoped as defined above.  I still think that\r
 100 > we need a better story for upgrades to the dump format in general, but\r
 101 > maybe this isn't the place to make that particular case.\r
 102 >\r
 103 >             --dkg\r
 104 \r
 105 I'm not sure what you have in mind, something more ambitious than the\r
 106 header added post 0.22?\r
 107 \r
 108 --=-=-=\r
 109 Content-Type: application/pgp-signature; name="signature.asc"\r
 110 \r
 111 -----BEGIN PGP SIGNATURE-----\r
 112 Version: GnuPG v1\r
 113 \r
 114 iQGcBAEBCAAGBQJXUX3pAAoJEPIClx2kp54sHwgL/3mH0iM4myE4hF6a7N4dgRJZ\r
 115 pHBSKknIlHU+5GJUc9gztdjKfTQZk/lUqPsW8ibh9hGzWyWvrsqZcSvlQr1tGnCD\r
 116 bx6K6qO4ES4hSDlysyiyCje/XKuFLGi9mD09DRxJjSL+Rlezy9SoLTiqG/JIhoB8\r
 117 6Q6pF1etof4l/ysm99+9u5ITO9gW3cc4x6xQpLPjeNLkFrmFfG9RBT/bsDhe2jq7\r
 118 Fyan4uXWCXICuTxnBP5uYrc1UMJPmaczxkEOPKCD9Af52o8sQIm74uYdUNYHuJIR\r
 119 0R+xMlQJX415WhMrjvnnCNpuWDQOZHKF2R68Dp4QXbIzTK05uPcKoBGAI7BE25Y4\r
 120 Vv1UefIyWp8s4KGbpLJRdbbu4eycqobT3WL9XwCtzrm58fyOu0g06YksKxrvjIN3\r
 121 yZm4/SvzUaD9JlfJOjRxNGSxKErRBNEYTtOo0ZXT1BOy2tG6jltU6YlcGghEcTmX\r
 122 x+/p/C7nQS6bfy4jqvYmnDDSq/uD28sHbmUYRlxtaQ==\r
 123 =RGB8\r
 124 -----END PGP SIGNATURE-----\r
 125 --=-=-=--\r