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
requestMetadata should take the md5 and sha256 with expected encodings. #852
Conversation
ac2da82
to
a31015c
Compare
build fail needs to be addressed. |
a31015c
to
3093a47
Compare
Fixed @poornas |
3093a47
to
d0f2bb3
Compare
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.
api-put-object-streaming.go
Outdated
// It is to indicate that *minio.Object implements io.ReaderAt. | ||
// and such a functionality is used in the subsequent code path. | ||
if isFile(reader) || !isObject(reader) && isReadAt(reader) { | ||
// Verify if reader is *minio.Object, if yes then use a sequential uploader. |
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.
The way the comment is split before and after the if-block is confusing. We could have comments describing how we pick sequential or parallel upload strategy in one place.
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.
Fixed
This is done for readability and also to better communicate the requirements of an API.
d0f2bb3
to
a1c555f
Compare
This is done for readability and also to better communicate
the requirements of an API.