[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