Skip to content
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

Deprecate listIncompleteUploads() and removeIncompleteUploads() #991

Conversation

balamurugana
Copy link
Member

Public methods in MinioClient does not leave incomplete uploads on
failures. Hence listIncompleteUploads() and
removeIncompleteUploads() are deprecated and will be removed later.

However protected methods listMultipartUploads(), listParts() and
abortMultipartUpload() in MinioClient are available by
inheritance for developers.

@balamurugana balamurugana force-pushed the Deprecate-listIncompleteUploads-and-removeIncomplete branch 3 times, most recently from 5d7ffe4 to 431eec8 Compare June 26, 2020 05:10
kannappanr
kannappanr previously approved these changes Jun 26, 2020
@BigUstad
Copy link
Contributor

@balamurugana Can you please re-run the jobs or look into the failures?

@balamurugana balamurugana force-pushed the Deprecate-listIncompleteUploads-and-removeIncomplete branch from 8dd5b5c to 0edb419 Compare July 2, 2020 04:54
kannappanr
kannappanr previously approved these changes Jul 3, 2020
Copy link
Contributor

@kannappanr kannappanr left a comment

Choose a reason for hiding this comment

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

LGTM

Public methods in MinioClient does not leave incomplete uploads on
failures. Hence `listIncompleteUploads()` and
`removeIncompleteUploads()` are deprecated and will be removed later.

However protected methods `listMultipartUploads()`, `listParts()` and
`abortMultipartUpload()` in MinioClient are available by
inheritance for developers.
@balamurugana balamurugana force-pushed the Deprecate-listIncompleteUploads-and-removeIncomplete branch from 0edb419 to 04aaf08 Compare July 6, 2020 16:41
@balamurugana
Copy link
Member Author

Already taken care by #992

@balamurugana balamurugana deleted the Deprecate-listIncompleteUploads-and-removeIncomplete branch July 7, 2020 08:26
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.

3 participants