f7/8f950046256fcb4cba80eee1588b052320424d

   1 Return-Path: <tomi.ollila@iki.fi>\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 91F0A431FC3\r
   6         for <notmuch@notmuchmail.org>; Sat,  4 Oct 2014 01:31:11 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id VkZ0Vr+ZbOC7 for <notmuch@notmuchmail.org>;\r
  16         Sat,  4 Oct 2014 01:31:06 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id AC2F9431FB6\r
  19         for <notmuch@notmuchmail.org>; Sat,  4 Oct 2014 01:31:06 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 2EB6D10008D;\r
  22         Sat,  4 Oct 2014 11:30:49 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Austin Clements <aclements@csail.mit.edu>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH 00/11] Add ghost messages and fix thread linking\r
  26 In-Reply-To: <1412345958-8278-1-git-send-email-aclements@csail.mit.edu>\r
  27 References: <1412345958-8278-1-git-send-email-aclements@csail.mit.edu>\r
  28 User-Agent: Notmuch/0.18.1+130~ga61922f (http://notmuchmail.org) Emacs/24.3.1\r
  29         (x86_64-unknown-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Sat, 04 Oct 2014 11:30:48 +0300\r
  34 Message-ID: <m2oatsz16v.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Sat, 04 Oct 2014 08:31:11 -0000\r
  50 \r
  51 On Fri, Oct 03 2014, Austin Clements <aclements@csail.mit.edu> wrote:\r
  52 \r
  53 > This series modifies our database representation of messages that have\r
  54 > been referenced by other messages, but for which we don't have the\r
  55 > message itself.  Currently, we store this information as Xapian\r
  56 > metadata, but this has several downsides for performance and\r
  57 > complexity and results in hard-to-fix thread linking bugs.  This patch\r
  58 > series implements "ghost messages", which replace this Xapian metadata\r
  59 > with Xapian documents that look and act very much like regular message\r
  60 > documents, but simply have no content.  This simplifies and speeds up\r
  61 > our thread linking algorithm and fixes the currently broken thread\r
  62 > linking test.\r
  63 >\r
  64 > Ghost messages also open up interesting future possibilities, such as\r
  65 > "pre-seeding" tags for messages that are not yet indexed.  This could\r
  66 > be used to make notmuch insert simpler and more robust, as part of tag\r
  67 > synchronization, and to improve nmbug's behavior when tags arrive\r
  68 > before messages.\r
  69 \r
  70 The code looks OK to me -- there are IMO some strange things but those\r
  71 don't change the status quo -- I did look a little past the diffs into\r
  72 the code to understand something...\r
  73 \r
  74 I am now having these patches applied to my 'home' notmuch and haven't yet\r
  75 seen anything strange there. In this setup I have 27 emails missing that\r
  76 nmbug expects there to be -- let's see if I can get ghost messages there.\r
  77 \r
  78 ... and tests pass, ran while writing the above part...\r
  79 \r
  80 I'll put these in use next week on one of my 'work' notmuch. There I have\r
  81 seen a problem that while I am on one thread, \r
  82 `notmuch-poll-and-refresh-buffer' picks new mail but suddenly this thread\r
  83 now has changed it's thread id :O -- making the search buffer go blank.\r
  84 I'll see whether it still happens with these (may be totally unrelated)\r
  85 and perhaps investigate further...\r
  86 \r
  87 \r
  88 Tomi\r
  89 \r