Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11918 )

Change subject: raft_consensus_nonvoter-itest: deflake a bit
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11918/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11918/1//COMMIT_MSG@22
PS1, Line 22: maybe the cstate's UUID becomes
            : empty for a little while
I'm not sure about this: as I see from the code in catalog_manager.cc, the 
message "Leader not yet ready to serve requests" corresponds to the case when 
cached leader term doesn't correspond to the term from the consensus state.

The fix looks good to me -- at least it should not make it worse.



--
To view, visit http://gerrit.cloudera.org:8080/11918
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8192bd669e7e309943ea82718dd715238d520bbd
Gerrit-Change-Number: 11918
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 13 Nov 2018 17:45:52 +0000
Gerrit-HasComments: Yes

Reply via email to