Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18842 )
Change subject: [tests] fix flakiness in TestTabletCopyEncryptedServers ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/18842/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18842/1//COMMIT_MSG@19 PS1, Line 19: This patch updates the assertion to allow both the COPYING and READY > couldn't we inject latency instead? I didn't try that option yet: just found that similar flakiness was fixed this way some time ago, so I though I'd simply use the same approach: https://github.com/apache/kudu/commit/54839984932bca0c0ba49cdd8fa199a5711e589e Let me know if you think injecting latency is the preferred way. -- To view, visit http://gerrit.cloudera.org:8080/18842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I22933cc9cb727711ee5fb45c811c2a759958fdfa Gerrit-Change-Number: 18842 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Sat, 20 Aug 2022 02:58:46 +0000 Gerrit-HasComments: Yes
