remove the "override lease duration" feature #1837

Open
opened 2012-10-30 23:43:24 +00:00 by zooko · 1 comment
zooko commented 2012-10-30 23:43:24 +00:00
Owner

It seems like the feature of "delete all shares whose renewal time is more than X time ago" is redundant with the feature of "delete all shares whose renewal time was before Y timestamp". See how those two are implemented on the #666 branch to see how redundant the are with each other. ☺ https://github.com/davidsarah/tahoe-lafs/commit/1d33b833703ef15d4071d1e965464f35a17ee5cf

The existence of more than one way to do it adds cognitive load to my poor, overworked brain. Please help me by eliminating one of them.

It seems like the feature of "delete all shares whose renewal time is more than X time ago" is redundant with the feature of "delete all shares whose renewal time was before Y timestamp". See how those two are implemented on the #666 branch to see how redundant the are with each other. ☺ <https://github.com/davidsarah/tahoe-lafs/commit/1d33b833703ef15d4071d1e965464f35a17ee5cf> The existence of more than one way to do it adds cognitive load to my poor, overworked brain. Please help me by eliminating one of them.
tahoe-lafs added the
code-storage
normal
defect
1.9.2
labels 2012-10-30 23:43:24 +00:00
tahoe-lafs added this to the undecided milestone 2012-10-30 23:43:24 +00:00
davidsarah commented 2012-10-30 23:46:54 +00:00
Author
Owner

+1. Let's document this feature as deprecated (and say how to express it in terms of cutoff-date).

+1. Let's document this feature as deprecated (and say how to express it in terms of cutoff-date).
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/trac-2024-07-25#1837
No description provided.