]> git.rkrishnan.org Git - tahoe-lafs/tahoe-lafs.git/commit
helper: remove timings["existence_check"], aka "Already-In-Grid Check"
authorBrian Warner <warner@lothar.com>
Tue, 22 May 2012 04:13:11 +0000 (21:13 -0700)
committerBrian Warner <warner@lothar.com>
Tue, 22 May 2012 04:13:11 +0000 (21:13 -0700)
commite60982c851adf2de23b4d6463703e7ea4e4388c7
tree804debe76b1873a3f07dc98a79f5907087309d43
parent393c0729de055b136e30b87f81153a79708e9f3e
helper: remove timings["existence_check"], aka "Already-In-Grid Check"

This measured how long the Helper took to do a filecheck before asking
for ciphertext. The "Contacting Helper" report includes both
existence_check and the client-helper RTT.

For non-overlapping uploads, it was being returned correctly. But when
multiple upload requests overlapped, and the file was not already in the
grid, the filecheck would only run once, and its existence_check time
would be reported for all uploaders (even if they didn't have to wait
for that time). Cleaning that up proved too difficult: the only correct
place to report this time is from the initial remote_upload_chk() call,
but the return value of that is too constrained to accomodate it in the
needs-upload case.

So I'm removing it altogether. Eventually I plan to add a proper
events/times field and record more data, including this check, in a form
that can be drawn on a nice zoomable timeline view.

Old clients talking to a new Helper (which doesn't supply the value)
will tolerate the loss (they'll just display an empty field on the web
view).
src/allmydata/immutable/offloaded.py
src/allmydata/interfaces.py
src/allmydata/web/status.py
src/allmydata/web/upload-results.xhtml
src/allmydata/web/upload-status.xhtml