-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clarify that servers should extend TLL for blobs referenced in FindMissingBlobs #113
Conversation
(ContentAddressableStorage.FindMissingBlobs): Clarify that servers should extend lifetime for blobs requested.
@@ -246,6 +246,9 @@ service ContentAddressableStorage { | |||
// Clients can use this API before uploading blobs to determine which ones are | |||
// already present in the CAS and do not need to be uploaded again. | |||
// | |||
// Servers SHOULD increase the TTLs of the referenced blobs if necessary and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: "TTL" sounds like an implementation detail. Would it make sense to change "TTLs" to "lifetimes" (both here and the GetActionResult comment that already exists in this doc)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GetActionResult seems to use TTLs? Generally I agree that we should standardize on one wording. I don't have a preference either way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we fix terminology in a separate PR? I would prefer to get the intent across regardless :).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, of course. This is just a "nitpick".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I prefer "lifetime" to "TTL" but agree that the terminology change can be made separately
(ContentAddressableStorage.FindMissingBlobs): Clarify that servers
should extend lifetime for blobs requested.