Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23954 )
Change subject: [consensus] report tablet ID with CheckOpInSequence in AdvanceCommittedIndex ...................................................................... [consensus] report tablet ID with CheckOpInSequence in AdvanceCommittedIndex To help in troubleshooting of WAL corruption issues, it makes sense adding the tablet's identifier into the log message emitted by PendingRounds::AdvanceCommittedIndex() when CheckOpInSequence() returns non-OK status. Given that corruption isn't expected, add EXPECT_FALSE() where appropriate. This changelist doesn't contain any functional modifications. Change-Id: I70df3ea624140d73c1ae4d3664d30b74dbfc91cb Reviewed-on: http://gerrit.cloudera.org:8080/23949 Tested-by: Kudu Jenkins Reviewed-by: Gabriella Lotz <[email protected]> Reviewed-by: Marton Greber <[email protected]> (cherry picked from commit 8a0c4bd44365c5ce81554796ae9b1f2c3444c410) Reviewed-on: http://gerrit.cloudera.org:8080/23954 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/consensus/pending_rounds.cc 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23954 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I70df3ea624140d73c1ae4d3664d30b74dbfc91cb Gerrit-Change-Number: 23954 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
