Re: [PATCH v3 0/6] Make Emacs search use sexp format
authorJameson Graef Rollins <jrollins@finestructure.net>
Sun, 2 Jun 2013 15:51:14 +0000 (08:51 +1700)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:55:19 +0000 (09:55 -0800)
43/1bb3c0808b51e49be3c1872ebfb433b30fc57e [new file with mode: 0644]

diff --git a/43/1bb3c0808b51e49be3c1872ebfb433b30fc57e b/43/1bb3c0808b51e49be3c1872ebfb433b30fc57e
new file mode 100644 (file)
index 0000000..32ec51d
--- /dev/null
@@ -0,0 +1,142 @@
+Return-Path: <jrollins@finestructure.net>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 1312A429E58\r
+       for <notmuch@notmuchmail.org>; Sun,  2 Jun 2013 08:51:33 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -2.3\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id 7sPSOnPf2kLs for <notmuch@notmuchmail.org>;\r
+       Sun,  2 Jun 2013 08:51:24 -0700 (PDT)\r
+Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
+       [131.215.239.19])\r
+       by olra.theworths.org (Postfix) with ESMTP id D7E88429E55\r
+       for <notmuch@notmuchmail.org>; Sun,  2 Jun 2013 08:51:24 -0700 (PDT)\r
+Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
+       by fire-doxen-postvirus (Postfix) with ESMTP id 5F71E2E50E39;\r
+       Sun,  2 Jun 2013 08:51:22 -0700 (PDT)\r
+X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
+Received: from finestructure.net (cpe-76-173-75-86.socal.res.rr.com\r
+       [76.173.75.86]) (Authenticated sender: jrollins)\r
+       by fire-doxen-submit (Postfix) with ESMTP id 715772E50E38;\r
+       Sun,  2 Jun 2013 08:51:17 -0700 (PDT)\r
+Received: by finestructure.net (Postfix, from userid 1000)\r
+       id E8C886176D; Sun,  2 Jun 2013 08:51:16 -0700 (PDT)\r
+From: Jameson Graef Rollins <jrollins@finestructure.net>\r
+To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v3 0/6] Make Emacs search use sexp format\r
+In-Reply-To: <87sj12yqyu.fsf@maritornes.cs.unb.ca>\r
+References: <1370047208-12785-1-git-send-email-amdragon@mit.edu>\r
+       <87sj12yqyu.fsf@maritornes.cs.unb.ca>\r
+User-Agent: Notmuch/0.15.2+155~g7fa0560 (http://notmuchmail.org) Emacs/24.3.1\r
+       (x86_64-pc-linux-gnu)\r
+Date: Sun, 02 Jun 2013 08:51:14 -0700\r
+Message-ID: <87r4gk8qa5.fsf@servo.finestructure.net>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/signed; boundary="=-=-=";\r
+       micalg=pgp-sha256; protocol="application/pgp-signature"\r
+Cc: tomi.ollila@iki.fi\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sun, 02 Jun 2013 15:51:33 -0000\r
+\r
+--=-=-=\r
+Content-Type: text/plain\r
+\r
+On Sat, Jun 01 2013, David Bremner <david@tethera.net> wrote:\r
+> Austin Clements <amdragon@MIT.EDU> writes:\r
+>\r
+>> This is v3 of id:1369934016-22308-1-git-send-email-amdragon@mit.edu.\r
+>> This tweaks the shell invocation as suggested by Tomi and fixes two\r
+>> comment typos pointed out by Mark.  It also adds a NEWS patch.  I'm\r
+>> going to go ahead and mark this ready because of Tomi's and Mark's\r
+>> reviews of v2.\r
+>\r
+> The first 5 I pushed. The NEWS patch has a conflict.\r
+\r
+I'm very happy to see the long-coming sexp handling working here.  Good\r
+work, folks, particularly to Austin for getting the awesome asynchronous\r
+processing stuff working.  Searches are now definitely noticeably\r
+faster.\r
+\r
+I am, however, seeing a couple of issues that we might want to address.\r
+\r
+* Killing a search buffer that is still in the process of being filled\r
+  causes errors to be thrown.  I'm seeing both of the following\r
+  intermittently:\r
+\r
+[Sun Jun  2 08:26:40 2013]\r
+notmuch exited with status killed\r
+command: notmuch search --format\=sexp --format-version\=1 --sort\=newest-first to\:jrollins\r
+exit signal: killed\r
+\r
+[Sun Jun  2 08:32:26 2013]\r
+notmuch exited with status hangup\r
+command: notmuch search --format\=sexp --format-version\=1 --sort\=newest-first to\:jrollins\r
+exit signal: hangup\r
+\r
+  This is somewhat understandable, as the notmuch binary exits with an\r
+  error if it hasn't finished dumping the output, but given how common\r
+  this particular scenario is I think we should try to avoid throwing\r
+  errors in this circumstance.  I wonder if we shouldn't just modify the\r
+  binary to not return non-zero if it was manually killed while\r
+  processing the output, or at least special-case the particular error\r
+  caused by manually killing the search.\r
+\r
+* The next thing I'm seeing is this:\r
+\r
+Opening input file: no such file or directory, /home/jrollins/tmp/nmerr5390CAY\r
+\r
+  I'm not exactly sure what causes this error, but it looks to me like\r
+  the temporary error file was removed before we were finished with it.\r
+\r
+* Finally, something happened that caused *12,000* of the following lines\r
+  to be sent to the *Notmuch errors* buffer:\r
+\r
+A Xapian exception occurred performing query: The revision being read has been discarded - you should call Xapian::Database::reopen() and retry the operation\r
+\r
+  Again, this was related to killing a search buffer that was still\r
+  being filled. I'm pretty sure the database was not modified during\r
+  this process.\r
+\r
+Let me know if I can help provide any more info.\r
+\r
+jamie.\r
+\r
+--=-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.12 (GNU/Linux)\r
+\r
+iQIcBAEBCAAGBQJRq2nyAAoJEO00zqvie6q8LJsP/2WvgF2Lsuqfue2TOlTOXby6\r
+k//nH/vTkHFC98JRGWv9jqvh1tJDQML9wERaEyyneabGTtFPGhUwJ+4P5FMPega3\r
+NnYFGfeW+fVAVEhv07zbSm8EQ99cZEor+z0J+dmsqCdntG8OPiGtracw0325I1w5\r
+lK8AlFcSNhUzsGnHBh2AQq3KePWXzKa101taMGJJ/bM+VvHRwqUdZapyS0vhfLrY\r
+Js6oRxwE4r0dirRM0n8ohu4ID6yqICvYqsNsP7igb36TqSNs9ttU0VNt1VvGEXDj\r
+dDU66KNCZgH9Zr0zWjAPAHbLoEZ6UduqWRcVvsnc5EbmDJq49Mg7mN8pisOgwImS\r
+lG0KIOnUfH1PPidy4jZzV1m50s8E0OxpQdyiFN+z3SXhgXT1lxnE6JkAJt9KXEk9\r
+rmZFlI3oAEcA15Nog90U87xdZx4FvYqy3hREkiqGUIHXQTMXbOZF/D2tI78p/GUF\r
+kxHOegbC+rsdfccPLr/76YTrqAn7sGVI4u7Cwr4/RkAKl0V8m62E4yQ0ZCFtyFch\r
+vWT2JNLb3HPkg4wGqiDoRIvfhXPIxxhj+YUIp6TC8RJqVgKHKHkUHoCxd/Hrhe/J\r
+DA7faRbTCPsvxdwXxF8gBLFrZmV3cAhzxkpeCXrl7itEeIL1yb8ieWlVERTFgsOb\r
+21xEmWO/iAi1iEpL2yQp\r
+=yi7g\r
+-----END PGP SIGNATURE-----\r
+--=-=-=--\r