From: Zooko O'Whielacronx Date: Sat, 3 Jan 2009 18:41:09 +0000 (-0700) Subject: immutable: further loosen the performance-regression test to allow up to 45 reads X-Git-Url: https://git.rkrishnan.org/simplejson/encoder.py.html?a=commitdiff_plain;h=7adf905b9f468b8e71599fbbc2097e77cb61cb0c;p=tahoe-lafs%2Ftahoe-lafs.git immutable: further loosen the performance-regression test to allow up to 45 reads This does raise the question of if there is any point to this test, since I apparently don't know what the answer *should* be, and whenever one of the buildbots fails then I redefine success. But, I'm about to commit a bunch of patches to implement checker, verifier, and repairer as well as to refactor downloader, and I would really like to know if these patches *increase* the number of reads required even higher than it currently is. --- diff --git a/src/allmydata/test/test_immutable.py b/src/allmydata/test/test_immutable.py index 9afaff39..dd4da451 100644 --- a/src/allmydata/test/test_immutable.py +++ b/src/allmydata/test/test_immutable.py @@ -441,11 +441,11 @@ class Test(ShareManglingMixin, unittest.TestCase): def _after_attempt(unused=None): after_download_reads = self._count_reads() # To pass this test, you are required to give up before reading all of the share - # data. Actually, we could give up sooner than 39 reads, but currently our download - # code does 39 reads. This test then serves as a "performance regression detector" + # data. Actually, we could give up sooner than 45 reads, but currently our download + # code does 45 reads. This test then serves as a "performance regression detector" # -- if you change download code so that it takes *more* reads, then this test will # fail. - self.failIf(after_download_reads-before_download_reads > 39, (after_download_reads, before_download_reads)) + self.failIf(after_download_reads-before_download_reads > 45, (after_download_reads, before_download_reads)) d.addCallback(_after_attempt) return d