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

Change subject: [consensus] KUDU-2181 Allow sending status msgs to FAILED peers
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16830/5/src/kudu/consensus/consensus_queue.cc
File src/kudu/consensus/consensus_queue.cc:

http://gerrit.cloudera.org:8080/#/c/16830/5/src/kudu/consensus/consensus_queue.cc@707
PS5, Line 707: peer_copy.wal_catchup_possible
Is this part is not exactly related to the changes in the context of 
multi-master automation tasks?  If so, maybe it's worth separating it into its 
own changelist for easier tracking and visibility?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I142c1dec442ec72c38c5be9d62cdf270e441d6e3
Gerrit-Change-Number: 16830
Gerrit-PatchSet: 5
Gerrit-Owner: Bankim Bhavsar <[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: Tue, 15 Dec 2020 22:00:34 +0000
Gerrit-HasComments: Yes

Reply via email to