Opened 3 years ago

Closed 3 years ago

#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
branch-diff, diff-cov, branch-cov, buildbot
Author: kaizhang


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 3 years ago by kaizhang

  • Status changed from new to assigned

comment:2 Changed 3 years ago by kaizhang

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

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

comment:3 Changed 3 years ago by kaizhang

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

comment:4 Changed 3 years 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 3 years 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.