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
objecter: minor cleanups #19994
objecter: minor cleanups #19994
Conversation
retest this please |
@@ -708,10 +708,9 @@ void Objecter::_send_linger_ping(LingerOp *info) | |||
o->target = info->target; | |||
o->should_resend = false; | |||
_send_op_account(o); | |||
MOSDOp *m = _prepare_osd_op(o); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for the posterity, it might not look like a refactor as it claims because the message's tid is set using o->tid before "o->tid" is updated. but please note, Objecter::_send_op()
always m->set_tid(op->tid)
before sending the message.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@runsisi since you are at this file, could you help remove the line of https://github.com/ceph/ceph/blob/master/src/osdc/Objecter.cc#L3309 ? as _prepare_osd_op()
should take care of this already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, removed it since _prepare_osd_op()
has set the updated m->tid
.
thanks~
Signed-off-by: runsisi <runsisi@zte.com.cn>
it's more intuitive and consistent Signed-off-by: runsisi <runsisi@zte.com.cn>
0c2c25e
to
fc02286
Compare
remove
Objecter::kick_requests
which is unused and remove default arg forObjecter::_send_op
Signed-off-by: runsisi runsisi@zte.com.cn