[tahoe-lafs-trac-stream] [tahoe-lafs] #1877: SFTP: use new eventual utilities from deferredutil
tahoe-lafs
trac at tahoe-lafs.org
Sat Nov 24 19:41:57 UTC 2012
#1877: SFTP: use new eventual utilities from deferredutil
-------------------------+-------------------------------------------------
Reporter: | Owner: davidsarah
davidsarah | Status: assigned
Type: defect | Milestone: 1.11.0
Priority: minor | Version: 1.9.2
Component: code- | Keywords: sftp deferred eventual cloud-
frontend | backend leasedb easy cleanup
Resolution: |
Launchpad Bug: |
-------------------------+-------------------------------------------------
Changes (by davidsarah):
* status: new => assigned
Old description:
> The cloud backend + leasedb merge adds several deferred utilities:
> [https://github.com/davidsarah/tahoe-
> lafs/commit/7671b3e33f0c91e764fb869175ff].
>
> {{{eventually_callback}}} and {{{eventually_errback}}} were modelled on
> the versions already in sftpd.py, with an error-reporting improvement.
> eventual_chain is an abstraction of the pattern
> {{{
> source.addCallbacks(eventually_callback(target),
> eventually_errback(target))
> }}}
> which is used twice in sftpd.py. So we should change sftpd to use these
> versions, once the cloud backend + leasedb merge has landed.
New description:
The cloud backend + leasedb merge adds several deferred utilities:
[https://github.com/davidsarah/tahoe-
lafs/commit/7671b3e33f0c91e764fb869175ff].
{{{eventually_callback}}} and {{{eventually_errback}}} were modelled on
the versions already in sftpd.py, with an error-reporting improvement.
{{{eventual_chain}}} is an abstraction of the pattern
{{{
source.addCallbacks(eventually_callback(target),
eventually_errback(target))
}}}
which is used twice in sftpd.py. So we should change sftpd to use these
versions, once the cloud backend + leasedb merge has landed.
--
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1877#comment:1>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list