Abhishek Chennaka has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21580 )

Change subject: [client] add ScanTokenStaleRaftMembershipTest
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/21580/1/src/kudu/client/scan_token-test.cc
File src/kudu/client/scan_token-test.cc:

http://gerrit.cloudera.org:8080/#/c/21580/1/src/kudu/client/scan_token-test.cc@1721
PS1, Line 1721: ASSERT_NE(new_leader_tsd->uuid(), leader_tsd->uuid());
> Why?
Ah, okay thanks for the explanation.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6ce3d549d4ab2502c58deae1250b49ba16bbc914
Gerrit-Change-Number: 21580
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 15 Jul 2024 22:26:56 +0000
Gerrit-HasComments: Yes

Reply via email to