Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14947 )
Change subject: [consensus] LeaderElection cleanup ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/14947/3/src/kudu/consensus/raft_consensus.h File src/kudu/consensus/raft_consensus.h: http://gerrit.cloudera.org:8080/#/c/14947/3/src/kudu/consensus/raft_consensus.h@461 PS3, Line 461: static const char* State_Name(State state); > This is a const function and should be tagged as such. This is a static function, so no const is applicable. http://gerrit.cloudera.org:8080/#/c/14947/3/src/kudu/consensus/raft_consensus.h@465 PS3, Line 465: static MonoDelta MinimumElectionTimeout(); > const was dropped. Here 'const' does not make much sense for the return value. Actually, many static code analysis tools would warn if adding 'const' for the return value here. -- To view, visit http://gerrit.cloudera.org:8080/14947 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic3163ff5c7a628431a145c11133c54e26b7165cd Gerrit-Change-Number: 14947 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Thu, 26 Dec 2019 22:39:07 +0000 Gerrit-HasComments: Yes
