Skip to content

Conversation

harshavardhana
Copy link
Member

Additionally also return the stat info of the original
object for fget_object.

Fixes #480
Fixes #481

Copy link
Member

@donatello donatello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, except for inaccuracies in comments in examples.

docs/API.md Outdated
```py

# Put an object 'myobject' with contents from '/tmp/otherobject'.
# Put an object 'myobject' with contents from '/tmp/otherobject', prints hex md5sum of the object if uploaded successfully.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not accurate - for multipart objects the etag != hex m5sum

docs/API.md Outdated

import os
# Put a file with default content-type.
# Put a file with default content-type, upon success prints hex md5sum of the object.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not accurate - for multipart objects the etag != hex m5sum

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops yes.

Additionally also return the stat info of the original
object for fget_object.

Fixes minio#480
Fixes minio#481
@harshavardhana harshavardhana merged commit 3450449 into minio:master Feb 23, 2017
@harshavardhana harshavardhana deleted the fix-fapi branch February 23, 2017 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants