75/7c4d04cf7c78ec6deedf974949f5c62b18e49b

   1 Return-Path: <amdragon@mit.edu>\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 842CB431FAF\r
   6         for <notmuch@notmuchmail.org>; Sun, 25 Nov 2012 19:29:12 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 fKGrvirsBoZr for <notmuch@notmuchmail.org>;\r
  16         Sun, 25 Nov 2012 19:29:11 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-7.mit.edu (DMZ-MAILSEC-SCANNER-7.MIT.EDU\r
  18         [18.7.68.36])\r
  19         by olra.theworths.org (Postfix) with ESMTP id C952C431FAE\r
  20         for <notmuch@notmuchmail.org>; Sun, 25 Nov 2012 19:29:11 -0800 (PST)\r
  21 X-AuditID: 12074424-b7fbe6d000003b02-b3-50b2e207325b\r
  22 Received: from mailhub-auth-2.mit.edu ( [18.7.62.36])\r
  23         by dmz-mailsec-scanner-7.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 77.C9.15106.702E2B05; Sun, 25 Nov 2012 22:29:11 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-2.mit.edu (8.13.8/8.9.2) with ESMTP id qAQ3T99E011765; \r
  27         Sun, 25 Nov 2012 22:29:10 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id qAQ3T7B2016560\r
  32         (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT);\r
  33         Sun, 25 Nov 2012 22:29:08 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.80)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1TcpNb-0005wO-0a; Sun, 25 Nov 2012 22:29:07 -0500\r
  37 Date: Sun, 25 Nov 2012 22:29:06 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: David Bremner <david@tethera.net>\r
  40 Subject: Re: [Patch v4 2/2] test: initial performance testing infrastructure\r
  41 Message-ID: <20121126032906.GP4562@mit.edu>\r
  42 References: <1353855745-11697-1-git-send-email-david@tethera.net>\r
  43         <1353855745-11697-3-git-send-email-david@tethera.net>\r
  44         <20121125214023.GO4562@mit.edu> <87obilut2h.fsf@zancas.localnet>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: text/plain; charset=us-ascii\r
  47 Content-Disposition: inline\r
  48 In-Reply-To: <87obilut2h.fsf@zancas.localnet>\r
  49 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  50 X-Brightmail-Tracker:\r
  51  H4sIAAAAAAAAA+NgFmpileLIzCtJLcpLzFFi42IRYrdT0WV/tCnAYNU7YYsbrd2MFtdvzmR2\r
  52         YPJ4tuoWs8eWQ++ZA5iiuGxSUnMyy1KL9O0SuDLa/zYxFjzgr+hbu5+pgXEVTxcjJ4eEgInE\r
  53         9Uvz2SFsMYkL99azgdhCAvsYJba2hXUxcgHZGxgltk89xALhXGSSmNlxhwmiagmjxKdjIV2M\r
  54         HBwsAqoSux/pg4TZBDQktu1fzghiiwCFr26bDDaUWUBa4tvvZrBWYQEfiXO7ZoHZvALaErcm\r
  55         fmOEmA+0eMOl42wQCUGJkzOfsEA0a0nc+PeSCWQXyKDl/zhAwpwCuhLbTi0FKxEVUJGYcnIb\r
  56         2wRGoVlIumch6Z6F0L2AkXkVo2xKbpVubmJmTnFqsm5xcmJeXmqRrrlebmaJXmpK6SZGcFC7\r
  57         qOxgbD6kdIhRgINRiYdX4timACHWxLLiytxDjJIcTEqivJfuA4X4kvJTKjMSizPii0pzUosP\r
  58         MUpwMCuJ8E5kAsrxpiRWVqUW5cOkpDlYlMR5r6fc9BcSSE8sSc1OTS1ILYLJynBwKEnwsj4E\r
  59         ahQsSk1PrUjLzClBSDNxcIIM5wEaLglSw1tckJhbnJkOkT/FqCglzqsEkhAASWSU5sH1wpLO\r
  60         K0ZxoFeEeTVBqniACQuu+xXQYCagwcnXN4IMLklESEk1MEbkq9vcebBv0pycr0+yzkkmWPe5\r
  61         b/3yoWdyLK9730uFZr//LaqTLu2++PX9pFOuOxl2dTw/pXeHt6bqYPB/jc1n3i+UmbjZLTrS\r
  62         v3rNrPbT+cXTr00LMNy23vvCUfdV4U57kxwWvq902vVDwPgS09cn8+67bfv9nlOi6lfrtl85\r
  63         6tJhB6539C9TYinOSDTUYi4qTgQAaPnSUxUDAAA=\r
  64 Cc: notmuch@notmuchmail.org\r
  65 X-BeenThere: notmuch@notmuchmail.org\r
  66 X-Mailman-Version: 2.1.13\r
  67 Precedence: list\r
  68 List-Id: "Use and development of the notmuch mail system."\r
  69         <notmuch.notmuchmail.org>\r
  70 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  72 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  73 List-Post: <mailto:notmuch@notmuchmail.org>\r
  74 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  75 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  76         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  77 X-List-Received-Date: Mon, 26 Nov 2012 03:29:12 -0000\r
  78 \r
  79 Quoth David Bremner on Nov 25 at  8:05 pm:\r
  80 > Austin Clements <amdragon@MIT.EDU> writes:\r
  81 > >> +add_email_corpus takes arguments "--small" and "--medium" for when you\r
  82 > >> +want smaller corpuses to check.\r
  83 > >\r
  84 > > "corpora"?\r
  85 > \r
  86 > reworded to say \r
  87 > \r
  88 > ,----\r
  89 > | add_email_corpus takes arguments "--small" and "--medium" for when you\r
  90 > | want smaller subsets of the corpus to check.\r
  91 > `----\r
  92 \r
  93 That's clearer.\r
  94 \r
  95 > >\r
  96 > > I'm a bit confused by this.  What happens if you don't specify --small\r
  97 > > or --medium?  Is the "large"/default corpus just the combined small\r
  98 > > and medium corpora?  Would be worth a comment, at least.\r
  99 > \r
 100 > Hopefully the README makes this clear(er) now?\r
 101 \r
 102 The README definitely helps.  Might still be worth a comment in the\r
 103 code since it took me some thinking to realize it would do something\r
 104 reasonable when given no argument.  Perhaps above the initial\r
 105 assignment of arg,\r
 106 \r
 107 # With no argument, use the entire (combined) corpus\r
 108 \r
 109 to acknowledge that this is a legitimate and intentional code path?\r
 110 \r
 111 > > This probably doesn't matter now, but I wonder if we want to unpack on\r
 112 > > first use to somewhere not test-specific and then cp -rl the corpus\r
 113 > > into the test directory.  I haven't tried unpacking the corpus yet,\r
 114 > > but if you're running tests repeatedly to compare results, or running\r
 115 > > more than one performance test, it seems like a full decompress and\r
 116 > > unpack could get onerous.\r
 117 > \r
 118 > Hmm. On my machine it is 10s for the copy versus 45s for a full\r
 119 > unpack. For some reason I tested with "cp -a" which is incredibly slow, \r
 120 > so I thought there was no loss. For comparison the basic test takes\r
 121 > about 10 minutes on the same machine.\r
 122 > \r
 123 > In any case this can wait until we have a second test file and a second\r
 124 > call to add_mail_corpus, adding caching now would not help.\r
 125 \r
 126 It would help (a little) if you run basic multiple times.  I think\r
 127 it's completely reasonable to leave it as is for now and see if\r
 128 caching would help down the road.\r