-
Notifications
You must be signed in to change notification settings - Fork 512
Sync APIs with S3 specification #1678
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
Sync APIs with S3 specification #1678
Conversation
7cb505b to
97cdc0a
Compare
a186c07 to
29c4a9b
Compare
shtripat
left a comment
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.
lgtm
anjalshireesh
left a comment
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.
LGTM, one minor nit and a clarification.
b8c0c94 to
ef2a2fe
Compare
anjalshireesh
left a comment
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.
LGTM
Co-authored-by: Shireesh Anjal <[email protected]> Co-authored-by: Sivanantham Gnanavel <[email protected]> Signed-off-by: Bala.FA <[email protected]>
bd60a4e to
3273c9f
Compare
sivanantham-gnanavel
left a comment
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.
LGTM
e9fbd85
Signed-off-by: Bala.FA <[email protected]>
e9fbd85 to
9185758
Compare
No description provided.