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

common/Finisher: Using queue(list<context*>) instead queue(context*). #8942

Merged
merged 1 commit into from Jan 4, 2017

Conversation

Projects
None yet
4 participants
@majianpeng
Member

majianpeng commented May 5, 2016

Avoid call many time queue(context*).

Signed-off-by: Jianpeng Ma jianpeng.ma@intel.com

@yuyuyu101

This comment has been minimized.

Show comment
Hide comment
@yuyuyu101

yuyuyu101 Sep 8, 2016

Member

rebase

Member

yuyuyu101 commented Sep 8, 2016

rebase

@liewegas liewegas added the bluestore label Dec 29, 2016

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Dec 29, 2016

Member

sorry, please rebase one more time? This looks good.

Member

liewegas commented Dec 29, 2016

sorry, please rebase one more time? This looks good.

@majianpeng

This comment has been minimized.

Show comment
Hide comment
@majianpeng

majianpeng Dec 30, 2016

Member

update

Member

majianpeng commented Dec 30, 2016

update

common/Finisher: Using queue(list<context*>) instead queue(context*).
Avoid call many time queue(context*).

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

@tchaikov tchaikov merged commit ee5d8b9 into ceph:master Jan 4, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment