Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23949 )
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]> --- M src/kudu/consensus/pending_rounds.cc 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Gabriella Lotz: Looks good to me, but someone else must approve Marton Greber: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23949 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I70df3ea624140d73c1ae4d3664d30b74dbfc91cb Gerrit-Change-Number: 23949 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
