Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16021 )
Change subject: [consensus] small cleanup on Peer::SignalRequest() ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/16021/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16021/1//COMMIT_MSG@12 PS1, Line 12: worst the ca > nit: the worst case Done http://gerrit.cloudera.org:8080/#/c/16021/1//COMMIT_MSG@14 PS1, Line 14: threads > nit: blocking other tasks? Done http://gerrit.cloudera.org:8080/#/c/16021/1/src/kudu/consensus/consensus_peers.h File src/kudu/consensus/consensus_peers.h: http://gerrit.cloudera.org:8080/#/c/16021/1/src/kudu/consensus/consensus_peers.h@197 PS1, Line 197: // Lock that protects Peer state changes, initialization, etc. > Could you update this to describe the new semantics of locking? E.g. seems Done -- To view, visit http://gerrit.cloudera.org:8080/16021 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I899aec0964bc404e6df3742bd48d0d049e52d900 Gerrit-Change-Number: 16021 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 03 Jun 2020 03:29:08 +0000 Gerrit-HasComments: Yes
