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: remove duplicated "commit_queued_for_journal_write" in OpTracker #21018
Conversation
b1bc69c
to
c2dec64
Compare
|
@@ -1609,8 +1609,6 @@ void FileJournal::submit_entry(uint64_t seq, bufferlist& e, uint32_t orig_len, | |||
assert(e.length() > 0); | |||
assert(e.length() < header.max_size); | |||
|
|||
if (osd_op) | |||
osd_op->mark_event("commit_queued_for_journal_write"); |
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.
hi @tchaikov would you mind taking a look? should we delete this event directly or use other event name? thank you
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.
i think we'd better remove this one. as their timestamp should always be identical.
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.
the title of the commit message looks like a bug report. please see https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#3-describe-your-changes for how to describe your changes.
c2dec64
to
cb1b5e3
Compare
@ashitakasam could you revise the title of your commit message so it describes your change instead of the bug symptom addressed by it? |
cb1b5e3
to
20dcb4d
Compare
20dcb4d
to
5e124a7
Compare
* src/os/filestore/FileJournal.cc: remove line 1612-1613 which will print "commit_queued_for_journal_write" twice. Fixes: http://tracker.ceph.com/issues/23440 Signed-off-by: ashitakasam <694240887@qq.com>
5e124a7
to
9bb1fb0
Compare
@tchaikov ping |
retest this please |
failed tests are |
Fixes: http://tracker.ceph.com/issues/23440
Signed-off-by: ashitakasam 694240887@qq.com