Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17120 )
Change subject: [client-test] added WriteWhileRestartingMultipleTabletServers ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/17120/1/src/kudu/client/client-test.cc File src/kudu/client/client-test.cc: http://gerrit.cloudera.org:8080/#/c/17120/1/src/kudu/client/client-test.cc@7912 PS1, Line 7912: for (auto mode : {KuduScanner::READ_LATEST, KuduScanner::READ_YOUR_WRITES}) { > At first I thought this could lead to flakiness because there could be a le Yes, there isn't any flakiness: it's LEADER_ONLY replica selection. If that's a former leader becomes stale, it has all the written rows because the same client is used to scan the rows as was use to write them. As for the original txn-related issue mentioned in the description of this patch, I guess the issue should be fixed once https://gerrit.cloudera.org/#/c/17037/10/src/kudu/integration-tests/txn_write_ops-itest.cc@1319 passes (I hope https://gerrit.cloudera.org/#/c/17127/ should fix it). -- To view, visit http://gerrit.cloudera.org:8080/17120 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I95d1456dea2e6e2bb7d8b0c5d05e95798098710d Gerrit-Change-Number: 17120 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 26 Feb 2021 03:09:38 +0000 Gerrit-HasComments: Yes
