d4/47331df6617a6771ca1ff875534fe5f10e36a3

   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 23CE3431FBD;\r
   6         Thu, 10 Dec 2009 21:13:09 -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 AGU-VliYYTt7; Thu, 10 Dec 2009 21:13:08 -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 389B4431FAE;\r
  13         Thu, 10 Dec 2009 21:13:08 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id 6ACB7254312; Thu, 10 Dec 2009 21:13:07 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: Kan-Ru Chen <kanru@kanru.info>, notmuch <notmuch@notmuchmail.org>\r
  18 In-Reply-To: <87r5r3joth.fsf@anar.kanru.info>\r
  19 References: <87r5r3joth.fsf@anar.kanru.info>\r
  20 Date: Thu, 10 Dec 2009 21:13:07 -0800\r
  21 Message-ID: <87638e3xgc.fsf@yoom.home.cworth.org>\r
  22 MIME-Version: 1.0\r
  23 Content-Type: multipart/signed; boundary="=-=-=";\r
  24         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  25 Subject: Re: [notmuch] Patch for xapian defect #250\r
  26 X-BeenThere: notmuch@notmuchmail.org\r
  27 X-Mailman-Version: 2.1.12\r
  28 Precedence: list\r
  29 List-Id: "Use and development of the notmuch mail system."\r
  30         <notmuch.notmuchmail.org>\r
  31 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  32         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  33 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  34 List-Post: <mailto:notmuch@notmuchmail.org>\r
  35 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  36 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  37         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  38 X-List-Received-Date: Fri, 11 Dec 2009 05:13:09 -0000\r
  39 \r
  40 --=-=-=\r
  41 \r
  42 On Thu, 10 Dec 2009 15:00:42 +0800, Kan-Ru Chen <kanru@kanru.info> wrote:\r
  43 > The termlist is already sorted, so this is the patch trying to minimize\r
  44 > the modification of database as suggested in the comment and Carl's\r
  45 > TODO file.\r
  46 \r
  47 Fantastic, Kan-Ru!\r
  48 \r
  49 > My poor profiling shows not much, but some improvement.\r
  50 \r
  51 Now you're just understating for sake of the pun. A 5-6x performance\r
  52 improvement looks great. And I see that as well in my testing:\r
  53 \r
  54 Before:\r
  55 \r
  56         $ time notmuch tag +foo tag:sent\r
  57         real 3m18.067s\r
  58         $ time notmuch tag -foo tag:sent\r
  59         real 3m12.940s\r
  60 \r
  61 After:\r
  62 \r
  63         $ time notmuch tag +foo tag:sent\r
  64         real 0m31.497s\r
  65         $ time notmuch tag -foo tag:sent\r
  66         real 0m28.722s\r
  67 \r
  68 I didn't flush the OS caches between runs, but a subsequent run of the\r
  69 "before" code still performed similarly slow:\r
  70 \r
  71         $ time notmuch tag +foo tag:sent\r
  72         real 3m7.172s\r
  73 \r
  74 And if I *had* used cold caches for every run the benefit of the patch\r
  75 would have looked even better.\r
  76 \r
  77 Anyway, we should get this upstream to the Xapian folks straight\r
  78 away. I expect they'll want to see a patch to the chert backend as well\r
  79 as the flint backend, (but fortunately the relevant code looks very\r
  80 similar if not identical).\r
  81 \r
  82 Thanks again,\r
  83 \r
  84 -Carl\r
  85 \r
  86 --=-=-=\r
  87 Content-Type: application/pgp-signature\r
  88 \r
  89 -----BEGIN PGP SIGNATURE-----\r
  90 Version: GnuPG v1.4.10 (GNU/Linux)\r
  91 \r
  92 iD8DBQFLIdTj6JDdNq8qSWgRAqquAKCB0fSF8muocnJgef2x7ys690YIsACdFjy7\r
  93 VxqxAhPKxdKXPpx0k1qm+tw=\r
  94 =7xLg\r
  95 -----END PGP SIGNATURE-----\r
  96 --=-=-=--\r