Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13461 )
Change subject: [java] Deflake TestAuthTokenReacquire.testBasicMasterOperations ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13461/2/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java: http://gerrit.cloudera.org:8080/#/c/13461/2/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java@135 PS2, Line 135: // TODO(KUDU-1537): Remove this workaround when table creation is exactly-once. master-stress-test also has this case: if (s.IsServiceUnavailable()) { // The client retried after the RPC timed out, and the retried RPC // arrived at the master while the table was still being created. // // TODO: Should be fixed with Exactly Once semantics, see KUDU-1537. continue; } Do we need to do the same here? -- To view, visit http://gerrit.cloudera.org:8080/13461 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifef0a254c6eb1cadeb54e9cef52f78c7ccc7b9c5 Gerrit-Change-Number: 13461 Gerrit-PatchSet: 2 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Thu, 30 May 2019 00:22:51 +0000 Gerrit-HasComments: Yes
