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

mgr: optimization some judgment and adjust the debug remove value in register_new_pgs #14046

Merged
merged 1 commit into from Apr 17, 2017

Conversation

Projects
None yet
3 participants
@songbaisen

songbaisen commented Mar 20, 2017

mgr: optimization some judgment and adjust the debug remove value in register_new_pgs

Signed-off-by: song baisen song.baisen@zte.com.cn

@songbaisen

This comment has been minimized.

songbaisen commented Mar 22, 2017

jenkins test this please

dout(15) << " had " << pgid << " from " << pg_map.pg_stat[pgid].reported_epoch << ":"
<< pg_map.pg_stat[pgid].reported_seq << dendl;
continue;
}
}

This comment has been minimized.

@jcsp

jcsp Apr 15, 2017

Contributor

Please remove this accidental whitespace at the end of the line

@jcsp

This comment has been minimized.

Contributor

jcsp commented Apr 15, 2017

This change looks sensible to me -- please could you just remove that little bit of whitespace, and also edit the commit message to start "mgr: "

songbaisen
mgr: optimization some judgment and adjust the debug remove value in …
…register_new_pgs

Signed-off-by: song baisen <song.baisen@zte.com.cn>

@songbaisen songbaisen changed the title from optimization some judgment and adjust the debug remove value in register_new_pgs to mgr: optimization some judgment and adjust the debug remove value in register_new_pgs Apr 17, 2017

@songbaisen

This comment has been minimized.

songbaisen commented Apr 17, 2017

@jcsp Thank you jscp. Done it.

@jcsp

jcsp approved these changes Apr 17, 2017

@jcsp jcsp merged commit 1247119 into ceph:master Apr 17, 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