54/066adb2dc20ea059dee61bc56171a1de785599

   1 Return-Path: <cworth@cworth.org>\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 olra.theworths.org (Postfix) with ESMTP id 62518431FBD;\r
   6         Mon, 21 Dec 2009 10:33:27 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id l7qr1eBlIeIY; Mon, 21 Dec 2009 10:33:26 -0800 (PST)\r
  11 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id AB3FA431FAE;\r
  13         Mon, 21 Dec 2009 10:33:26 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id 53070254306; Mon, 21 Dec 2009 10:33:26 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: James Westby <jw+debian@jameswestby.net>, notmuch@notmuchmail.org\r
  18 In-Reply-To: <1261186149-24078-1-git-send-email-jw+debian@jameswestby.net>\r
  19 References: <871virzzjy.fsf@yoom.home.cworth.org>\r
  20         <1261186149-24078-1-git-send-email-jw+debian@jameswestby.net>\r
  21 Date: Mon, 21 Dec 2009 10:33:21 -0800\r
  22 Message-ID: <87vdg0npn2.fsf@yoom.home.cworth.org>\r
  23 MIME-Version: 1.0\r
  24 Content-Type: multipart/signed; boundary="=-=-=";\r
  25         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  26 Subject: Re: [notmuch] [PATCH] Reindex larger files that duplicate ids we\r
  27         have\r
  28 X-BeenThere: notmuch@notmuchmail.org\r
  29 X-Mailman-Version: 2.1.12\r
  30 Precedence: list\r
  31 List-Id: "Use and development of the notmuch mail system."\r
  32         <notmuch.notmuchmail.org>\r
  33 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  34         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  35 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  36 List-Post: <mailto:notmuch@notmuchmail.org>\r
  37 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  38 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  40 X-List-Received-Date: Mon, 21 Dec 2009 18:33:27 -0000\r
  41 \r
  42 --=-=-=\r
  43 \r
  44 Hi James,\r
  45 \r
  46 I just got to a point in my outstanding rework where I thought it would\r
  47 make sense to pull this patch series in, (I'm adding support for storing\r
  48 multiple filenames in a single mail document).\r
  49 \r
  50 I took a closer look at this series, and I think it's still independent,\r
  51 so I'll finish up what I'm doing and then add this series on top later.\r
  52 \r
  53 But I can at least answer some of the questions you asked for now:\r
  54 \r
  55 >   Does the re-indexing replace the old terms?\r
  56 \r
  57 Before this patch, there's there's not yet any "re-indexing" in\r
  58 notmuch. So we'll basically need to think about what we want to do here.\r
  59 \r
  60 As this patch is written, (just calling into the existing _index_file\r
  61 function), the re-indexing only adds new terms, (and doesn't delete\r
  62 any). That's probably correct. We're using file size as an heuristic\r
  63 that the larger file is a superset of the smaller file, but it doesn't\r
  64 guarantee that the smaller file doesn't contain any unique terms. So I'd\r
  65 be extremely hesitant to drop any terms here.\r
  66 \r
  67 >                                               In the case\r
  68 >   where you had a collision with different text this could\r
  69 >   make a search return mails that don't contain that text.\r
  70 >   I don't think it's a big issue though, even if that is the\r
  71 >   case.\r
  72 \r
  73 That's correct. As mentioned in a previous thread, this is likely only a\r
  74 big issue in the face of deliberate message-ID spoofing or so. In that\r
  75 thread we talked about some ideas for mitigating that. But I don't think\r
  76 we need to solve that problem before applying this patch series.\r
  77 \r
  78 -Carl\r
  79 \r
  80 --=-=-=\r
  81 Content-Type: application/pgp-signature\r
  82 \r
  83 -----BEGIN PGP SIGNATURE-----\r
  84 Version: GnuPG v1.4.10 (GNU/Linux)\r
  85 \r
  86 iD8DBQFLL79x6JDdNq8qSWgRAs/9AKCZ+RWr9C2bkxVCrLeqau6L+2psigCfYcO9\r
  87 7WG+Tis8W9qhm0g7oESvOT8=\r
  88 =wtON\r
  89 -----END PGP SIGNATURE-----\r
  90 --=-=-=--\r