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.

Sign up for GitHub

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

Jump to bottom

object/ibmcos,ks3: check prefix and marker with returned key from S3 #3242

Closed
wants to merge 1 commit into from
Closed

object/ibmcos,ks3: check prefix and marker with returned key from S3 #3242

wants to merge 1 commit into from

Conversation

zhijian-pro
Copy link
Contributor

ref #2040

@davies
Copy link
Contributor

davies commented Feb 16, 2023

ibmcos and ks3 already guarantee the order, we so don't need to add this check.

btw, we will loose the requirement on order, so we can do this later.

@davies davies closed this Feb 16, 2023
@zhijian-pro zhijian-pro deleted the check_s3_prefix branch May 15, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davies davies Awaiting requested review from davies

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zhijian-pro @davies