Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/21580 )
Change subject: [client] add ScanTokenStaleRaftMembershipTest ...................................................................... Patch Set 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_LT(init_location_requests, NumGetTableLocationsRequests()) nit: Can we do a more tighter ASSERT_EQ on init_location_requests + 1 and NumGetTableLocationsRequests()? -- 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: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 12 Jul 2024 17:43:40 +0000 Gerrit-HasComments: Yes
