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

rgw: rgw_file skip policy read for virtual components #16034

Merged
merged 2 commits into from Jul 11, 2017

Conversation

Projects
None yet
2 participants
@guihecheng
Contributor

guihecheng commented Jun 30, 2017

virtual components that do not have backing objects:

  • virtual directory
  • parent directory("..")
    we should skip reading s3 policy for them.

Gui Hecheng added some commits Jun 29, 2017

Gui Hecheng
rgw_file: skip stat_leaf for ".." after readdir
Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
Gui Hecheng
rgw_file: skip object policy read for ListBucket operations
We don't need to read read object policy for a ListBucket operation
(RGWReaddir, RGWRMdirCheck, RGWStatLeaf).
Also, there are "virtual" directories that do not have a corresponding
object, read object policy on them results in ENOENT for RGWStatLeaf.

Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
@mattbenjamin

This comment has been minimized.

Contributor

mattbenjamin commented Jun 30, 2017

@guihecheng interesting, I need to read and experiment; I guess could be a big speedup

@mattbenjamin

lgtm, verified

@mattbenjamin mattbenjamin merged commit 3613da5 into ceph:master Jul 11, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
default Build finished.
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