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

mimic: rgw: allow radosgw-admin to list bucket w --allow-unordered #28096

Merged
merged 1 commit into from May 15, 2019

Conversation

Projects
None yet
3 participants
@ivancich
Copy link
Member

commented May 14, 2019

https://tracker.ceph.com/issues/39731


Presently the radosgw-admin bucket list --bucket=<bucket> lists the
objects in lexical order. This can be an expensive operation since
objects are not stored in bucket index shards in order and a selection
sort process is done across all bucket index shards.

By allowing the user to add the "--allow-unordered" command-line flag,
a more efficient bucket listing is enabled. This is particularly
important for buckets with a large number of objects.

Backport Tracker: https://tracker.ceph.com/issues/39731

rgw: allow radosgw-admin to list bucket w --allow-unordered
Presently the `radosgw-admin bucket list --bucket=<bucket>` lists the
objects in lexical order. This can be an expensive operation since
objects are not stored in bucket index shards in order and a selection
sort process is done across all bucket index shards.

By allowing the user to add the "--allow-unordered" command-line flag,
a more efficient bucket listing is enabled. This is particularly
important for buckets with a large number of objects.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
(cherry picked from commit 12452c4)

@ivancich ivancich requested a review from cbodley May 14, 2019

@ivancich

This comment has been minimized.

Copy link
Member Author

commented May 14, 2019

@yuriw Is it possible to get this into the upcoming mimic?

@smithfarm smithfarm added this to the mimic milestone May 15, 2019

@ivancich

This comment has been minimized.

Copy link
Member Author

commented May 15, 2019

@ivancich ivancich requested a review from adamemerson May 15, 2019

@adamemerson
Copy link
Contributor

left a comment

I long for this patch! With an all consuming, fiery passion.

Merge this PR soon, or watch the world devoured by the flames left in my wake-- I'll burn through you.

(Well, okay, maybe I don't long for it that much, but it's a good patch.)

@ivancich ivancich merged commit 2bb04a3 into ceph:mimic May 15, 2019

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.