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
ASAP call oncommit callback function to make OSDOpReply firstly reply. #13989
Conversation
@majianpeng your first commit has already been pulled by me. |
@liupan1111 . Ok, i'll remove this. |
df66659
to
bd02cde
Compare
Could you modify your pr title, and make it more meaningful? |
bd02cde
to
b8e70c4
Compare
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
if (op->done()) { | ||
assert(!op->on_commit && !op->on_applied); | ||
in_progress_ops.erase(op->tid); | ||
if (op->waiting_for_commit.empty()) { |
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.
@majianpeng NIT tabs left
assert(!op->on_commit && !op->on_applied); | ||
in_progress_ops.erase(op->tid); | ||
if (op->waiting_for_applied.empty()) { | ||
assert(!op->on_applied); |
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.
@majianpeng NIT tab left
Make op_commit->complete ASAP be called because callback of op_commit send reply to client.