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
rgw: update outdated debug func name #17440
Conversation
dfa852e
to
ab71a98
Compare
@joscollin @cbodley mind to merge? |
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.
Looks good to me.
src/rgw/rgw_rados.cc
Outdated
@@ -10645,13 +10645,13 @@ int RGWRados::Object::Read::iterate(int64_t ofs, int64_t end, RGWGetDataCB *cb) | |||
while (!done) { | |||
r = data->wait_next_io(&done); | |||
if (r < 0) { | |||
dout(10) << "get_obj_iterate() r=" << r << ", canceling all io" << dendl; | |||
dout(10) << __func__ << " r=" << r << ", cancelling all io" << dendl; |
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.
nit: canceling
is also a correct spelling, so i'd prefer not to change it
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.
@cbodley canceling
is American and cancelling
is British. So for people on the Eastern side of the world cancelling
looks correct :-D
https://english.stackexchange.com/questions/42745/canceling-or-cancelling
I think, let's follow American, as cbodley suggests.
Signed-off-by: Jiaying Ren <jiaying.ren@umcloud.com>
ab71a98
to
18b5482
Compare
Jenkins, retest this please. |
No description provided.