Closed
Bug 550586
Opened 15 years ago
Closed 14 years ago
Allow records to mark how long they should be kept (ttl)
Categories
(Cloud Services Graveyard :: Server: Sync, defect)
Cloud Services Graveyard
Server: Sync
Tracking
(Not tracked)
RESOLVED
FIXED
2.0
People
(Reporter: Mardak, Assigned: telliott)
References
Details
(Keywords: push-needed)
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
tarek
:
review+
|
Details | Diff | Splinter Review |
This could be used for the client to hint to the server that a record can/should be deleted after some time.
|
Assignee | |
Comment 1•15 years ago
|
||
I think this should be a X-Record-Keep-Until header that applies to all objects in the PUT/POST. Objections?
|
Reporter | |
Comment 2•15 years ago
|
||
The value for ttl will be sent as part of the payload of the WBOs in seconds until it may be removed.
Summary: Allow records to mark how long they should be kept → Allow records to mark how long they should be kept (ttl)
|
|
Assignee | |
Comment 4•14 years ago
|
||
This handles everything but the background cleanup script that we'll need to purge old ttls
Attachment #469254 -
Flags: review?(tarek)
|
||
Comment 5•14 years ago
|
||
There's a typo in your patch, that breaks an API:
$sth->bindParam(':username', time());
That's in weave_storage/mysql.php get_collection_storage_totals
I think we should to have some minimal functional tests for the quota code in the PHP before it goes live, to exercise it a bit.
|
|
Assignee | |
Comment 6•14 years ago
|
||
hah, good catch. That was in the quota code and I happened to have those tests off on my box.
Attachment #469254 -
Attachment is obsolete: true
Attachment #469497 -
Flags: review?(tarek)
Attachment #469254 -
Flags: review?(tarek)
|
|
||
Updated•14 years ago
|
Attachment #469497 -
Flags: review?(tarek) → review+
|
|
Assignee | |
Comment 7•14 years ago
|
||
|
||
Comment 8•14 years ago
|
||
Can we see a README update, or some other way of knowing what type ttl should be in the database schema so that we can still setup our own servers?
Thank you.
|
|
Assignee | |
Comment 9•14 years ago
|
||
I'm prepping a post on that. It's an int(11).
Will update the README.
|
||
Updated•2 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•