-
Notifications
You must be signed in to change notification settings - Fork 372
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 timeout and retries #323
Clarify timeout and retries #323
Conversation
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.
👍
spec.md
Outdated
|
||
Any of the RPCs defined in this spec MAY timeout and MAY be retried. | ||
The CO MAY chooses the maximum time it is willing to wait for a call | ||
and how many time it retries (these values are not a negotiated between |
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.
negotiation
spec.md
Outdated
plugin and CO). Idempotency requirements ensure that the call continues | ||
where it left off when retried. The only way to cancel a call is to | ||
issue a "negation" call if one exists (for example, issue an "unpublish" | ||
call for a pending "publish" operation, or a "delete" call for a pending |
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.
Use exact names: publish
-> ControllerUnpublishVolume
Don't even mention delete
spec.md
Outdated
where it left off when retried. The only way to cancel a call is to | ||
issue a "negation" call if one exists (for example, issue an "unpublish" | ||
call for a pending "publish" operation, or a "delete" call for a pending | ||
"create" operation, etc.). |
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.
Add a comment about "sometimes CO may not be able to recover".
1fa3541
to
b18b1ae
Compare
b18b1ae
to
3a642ac
Compare
Closes #128