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: fix reqid assignment for reply messages in OpRequest #17060
osd: fix reqid assignment for reply messages in OpRequest #17060
Conversation
b431da8
to
30199ed
Compare
src/osd/OpRequest.cc
Outdated
@@ -38,6 +39,8 @@ OpRequest::OpRequest(Message *req, OpTracker *tracker) : | |||
reqid = static_cast<MOSDSubOp*>(req)->reqid; | |||
} else if (req->get_type() == MSG_OSD_REPOP) { | |||
reqid = static_cast<MOSDRepOp*>(req)->reqid; | |||
} else if (req->get_type() == MSG_OSD_REPOPREPLY) { | |||
reqid = static_cast<MOSDRepOpReply*>(req)->reqid; |
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.
What about MOSDSubOpReply
?
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.
It would have been a better design if reqid
was private as it is in MOSDOp
. Don't know why it is not.
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.
Done, thanks.
919a06d
to
4186d9e
Compare
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
retest this please |
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.
LGTM
Signed-off-by: Yingxin Cheng yingxin.cheng@intel.com