#6720 enhancement closed fixed (fixed)

DeferredFilesystemLock should return a cancellable Deferred

Reported by: glyph Owned by: kaizhang
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/deferredfilesystemlock-cancellation-6720
(diff, github, buildbot, log)
Author: kaizhang Launchpad Bug:

Description

It already implements its own timeout internally; it would be nice if you could trigger this in advance. (Maybe this could even be refactored to use generic timeout support!)

Change History (5)

comment:1 Changed 13 months ago by kaizhang

  • Status changed from new to assigned

comment:2 Changed 13 months ago by kaizhang

  • Author set to kaizhang
  • Branch set to branches/deferredfilesystemlock-cancellation-6720

(In [39907]) Branching to 'deferredfilesystemlock-cancellation-6720'

comment:3 Changed 13 months ago by kaizhang

  • Keywords review added
  • Owner kaizhang deleted
  • Status changed from assigned to new

comment:4 Changed 12 months ago by tom.prince

  • Keywords review removed
  • Owner set to kaizhang

Thanks. One minor point of grammar: "When cancelling" not "When cancel". Please merge after fixing that.

comment:5 Changed 11 months ago by itamarst

  • Resolution set to fixed
  • Status changed from new to closed

(In [40558]) Merge deferredfilesystemlock-cancellation-6720: DeferredFilesystemLock supports Deferred cancellation.

Author: kaizhang
Reviewer: tom.prince
Fixes: #6720

The Deferred returned by twisted.internet.defer.DeferredFilesystemLock.deferUntilLocked can now be cancelled.

Note: See TracTickets for help on using tickets.