0c/d006bb6ee6dc76ede87689c236cbb59a949ab3

   1 Return-Path: <ethan.glasser.camp@gmail.com>\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 CB42F431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon, 15 Oct 2012 17:13:07 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id yYOY5iwFZpo9 for <notmuch@notmuchmail.org>;\r
  17         Mon, 15 Oct 2012 17:13:05 -0700 (PDT)\r
  18 Received: from mail-qa0-f53.google.com (mail-qa0-f53.google.com\r
  19         [209.85.216.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id B6660431FAE\r
  22         for <notmuch@notmuchmail.org>; Mon, 15 Oct 2012 17:13:05 -0700 (PDT)\r
  23 Received: by mail-qa0-f53.google.com with SMTP id s11so2163371qaa.5\r
  24         for <notmuch@notmuchmail.org>; Mon, 15 Oct 2012 17:13:05 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type;\r
  28         bh=t8EmuOexxqL8YFO6aVX4zN4D0lw/eaVpkR1H7q3exH4=;\r
  29         b=MoJZj9Dr+pBL9MQneIwxvkgyZs4RhAs+A6oaGoW4mZhA5nmE2HNVhWN2gB0g3q01pH\r
  30         KVLCklvKSbcVyFA2m7G3CV17KDw6psHNlYlnH4TZKcDCHd1CPID0jiQ8cqEVK0n1KQKH\r
  31         nFQeZlRTd5MXGbt0FZNQ6DfaJR1I4EKkmiKzGOxkZ3OnXEs/rxDCoovRqGpn0n+sDY66\r
  32         OdhKnkHerDRMjeKp8I3TXAh50OHA9bMfHADWQ8EdDqlyVHufGEc/3r+pQEOpOl0W/aMH\r
  33         9m/FK12r7R3T+T+8CJ9D9MPkoLVHmnKzxOSCLICTlTDEXXS+y0Aew2y+BqyOpNDVylw/\r
  34         1FMg==\r
  35 Received: by 10.224.107.5 with SMTP id z5mr23458461qao.41.1350346384972;\r
  36         Mon, 15 Oct 2012 17:13:04 -0700 (PDT)\r
  37 Received: from smtp.gmail.com (p70-80.acedsl.com. [66.114.70.80])\r
  38         by mx.google.com with ESMTPS id cz8sm11134483qab.21.2012.10.15.17.13.03\r
  39         (version=TLSv1/SSLv3 cipher=OTHER);\r
  40         Mon, 15 Oct 2012 17:13:04 -0700 (PDT)\r
  41 From: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>\r
  42 To: Sascha Silbe <sascha-ml-reply-to-2012-3@silbe.org>,\r
  43         notmuch <notmuch@notmuchmail.org>\r
  44 Subject: Re: [PATCH 3/3] new: don't read unchanged directories from disk\r
  45 In-Reply-To: <1340555366-25891-4-git-send-email-sascha-pgp@silbe.org>\r
  46 References: <1340555366-25891-1-git-send-email-sascha-pgp@silbe.org>\r
  47         <1340555366-25891-4-git-send-email-sascha-pgp@silbe.org>\r
  48 User-Agent: Notmuch/0.14+45~g6ea9330 (http://notmuchmail.org) Emacs/23.3.1\r
  49         (x86_64-pc-linux-gnu)\r
  50 Date: Mon, 15 Oct 2012 20:13:01 -0400\r
  51 Message-ID: <87y5j78eb6.fsf@betacantrips.com>\r
  52 MIME-Version: 1.0\r
  53 Content-Type: text/plain; charset=us-ascii\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Tue, 16 Oct 2012 00:13:07 -0000\r
  67 \r
  68 Sascha Silbe <sascha-pgp@silbe.org> writes:\r
  69 \r
  70 > Previously, notmuch new listed all directories on disk, even if they\r
  71 > were unchanged from the state recorded in the database. This could take\r
  72 > a huge amount of time for large numbers of mails as it would list each\r
  73 > individual mail.\r
  74 >\r
  75 > By iterating over the subdirectories recorded in the database we can\r
  76 > avoid accessing the file system for each unchanged directory. If the\r
  77 > modification time does not match we fall back to a full file system scan\r
  78 > so new subdirectories will get picked up and scanned recursively.\r
  79 \r
  80 Hi! Just going through the patch queue. These patches look fine to me,\r
  81 but I think architecturally, notmuch needs to decide what it wants to do\r
  82 -- if it wants to optimize directory-rescanning, or if it wants to\r
  83 provide a better way to just add files known to be new, and let mail\r
  84 delivery hook into that system. I'd prefer the second, but I think it's\r
  85 OK to merge working code that does the first in the meantime. The\r
  86 perfect is the enemy of the good.\r
  87 \r
  88 Austin raised some concerns on this patch set that I'm not sure were\r
  89 completely addressed. For my part, it seems intuitive that a stat()\r
  90 would be faster than a scandir() and so I would have no problem with\r
  91 merging this series.\r
  92 \r
  93 Ethan\r