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

os/bluestore: generate same onode extent-shard keys in a more efficient way #12681

Merged
merged 1 commit into from Jan 18, 2017

Conversation

Projects
None yet
3 participants
@xiexingguo
Member

xiexingguo commented Dec 27, 2016

No description provided.

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Dec 27, 2016

Member

The kv.h stuff is switched around by #12651... do you mind rebasing on top of that?

Member

liewegas commented Dec 27, 2016

The kv.h stuff is switched around by #12651... do you mind rebasing on top of that?

@xiexingguo

This comment has been minimized.

Show comment
Hide comment
@xiexingguo

xiexingguo Dec 27, 2016

Member

do you mind rebasing on top of that?

Never, will do.

Member

xiexingguo commented Dec 27, 2016

do you mind rebasing on top of that?

Never, will do.

Show outdated Hide outdated src/os/CMakeLists.txt Outdated
Show outdated Hide outdated src/os/bluestore/kv.h Outdated
Show outdated Hide outdated src/os/bluestore/kv.cc Outdated
Show outdated Hide outdated src/os/bluestore/BlueStore.cc Outdated
Show outdated Hide outdated src/os/bluestore/BlueStore.cc Outdated
@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Jan 15, 2017

Member

Please rebase now that #12651 has merged

Member

liewegas commented Jan 15, 2017

Please rebase now that #12651 has merged

@xiexingguo

This comment has been minimized.

Show comment
Hide comment
@xiexingguo

xiexingguo Jan 16, 2017

Member

updated and local tested:

==> rm -r bluestore.test_temp_dir
[       OK ] ObjectStore/StoreTest.OnodeSizeTracking/2 (5515 ms)
[----------] 67 tests from ObjectStore/StoreTest (4764767 ms total)

[----------] Global test environment tear-down
[==========] 67 tests from 1 test case ran. (4764767 ms total)
[  PASSED  ] 67 tests.
Member

xiexingguo commented Jan 16, 2017

updated and local tested:

==> rm -r bluestore.test_temp_dir
[       OK ] ObjectStore/StoreTest.OnodeSizeTracking/2 (5515 ms)
[----------] 67 tests from ObjectStore/StoreTest (4764767 ms total)

[----------] Global test environment tear-down
[==========] 67 tests from 1 test case ran. (4764767 ms total)
[  PASSED  ] 67 tests.
os/bluestore: generate same onode extent-shard keys in a more efficie…
…nt way

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@ifed01

ifed01 approved these changes Jan 16, 2017

@liewegas liewegas merged commit 01da856 into ceph:master Jan 18, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@xiexingguo xiexingguo deleted the xiexingguo:xxg-wip-bluestore-1227 branch Jan 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment