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

[delete] bench object from listing the bucket #290

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

fatpat
Copy link
Contributor

@fatpat fatpat commented Dec 6, 2023

backport listing feature from get to delete

when doing a delete bench, during the prepare phase, instead of putting objects in the bucket use the objects already in there. This can be used to bench a specific use case with an already filled in bucket.

New options:
--list-existing to activate the listing bucket (current
behaviour remains)
--list-flat: disable recursive listing

Copy link
Collaborator

@klauspost klauspost left a comment

Choose a reason for hiding this comment

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

Just some minor things.

If/when you want to do another one, we should see if we can generalize the code. I feel like almost everything in if d.ListExisting { could be generalized.

pkg/bench/delete.go Outdated Show resolved Hide resolved
pkg/bench/delete.go Outdated Show resolved Hide resolved
@fatpat
Copy link
Contributor Author

fatpat commented Jan 18, 2024

If/when you want to do another one, we should see if we can generalize the code. I feel like almost everything

I agree, were should I put the factorized code ?

Copy link
Collaborator

@klauspost klauspost left a comment

Choose a reason for hiding this comment

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

LGTM

@fatpat fatpat force-pushed the delete-listing branch 2 times, most recently from 5c6164d to 98d1954 Compare January 31, 2024 18:40
backport listing feature from get to delete

when doing a delete bench, during the prepare phase, instead of putting
objects in the bucket use the objects already in there. This can be used
to bench a specific use case with an already filled in bucket.

New options:
--list-existing to activate the listing bucket (current
  behaviour remains)
--list-flat: disable recursive listing
@klauspost klauspost merged commit 145d871 into minio:master Feb 26, 2024
8 of 9 checks passed
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