Alexey Serbin has uploaded this change for review. ( 
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)
---
M src/kudu/consensus/pending_rounds.cc
1 file changed, 8 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/23954/1
--
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: newchange
Gerrit-Change-Id: I70df3ea624140d73c1ae4d3664d30b74dbfc91cb
Gerrit-Change-Number: 23954
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to