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

osd: several small cleanups #7055

Merged
merged 4 commits into from Jan 13, 2016
Merged

osd: several small cleanups #7055

merged 4 commits into from Jan 13, 2016

Conversation

xiexingguo
Copy link
Member

No description provided.

@xiexingguo xiexingguo changed the title ReplicatedPG: fix unhandled result code and simplify result handing logic ReplicatedPG: simplify result handing logic a little bit Dec 25, 2015
It is going to fall through to fail lable anyway.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
the input parameter pg is not needed any more.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo xiexingguo changed the title ReplicatedPG: simplify result handing logic a little bit OSD: small cleanups Dec 26, 2015
@xiexingguo
Copy link
Member Author

@tchaikov Some simple and small cleanups. I would really appreciate it if you take a look at. Thanks.

@xiexingguo
Copy link
Member Author

@tchaikov Ping~

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@tchaikov
Copy link
Contributor

lgtm with a rados qa run.

liewegas added a commit that referenced this pull request Jan 13, 2016
osd: several small cleanups

Reviewed-by: Kefu Chai <kchai@redhat.com>
@liewegas liewegas merged commit 344d4c6 into ceph:master Jan 13, 2016
@xiexingguo xiexingguo deleted the xxg-wip-14185 branch January 13, 2016 04:19
@ghost ghost changed the title OSD: small cleanups osd: several small cleanups Feb 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants