Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7138
Change subject: KUDU-2037 fix flake in ts_recovery-itest ...................................................................... KUDU-2037 fix flake in ts_recovery-itest Fixed flake in TsRecoveryITest.TestRestartWithOrphanedReplicates scenario. The write operation timeout was set to 100ms, and for a ASAN/TSAN build that was under the reasonable minimum to successfully complete the majority of write operations. Prior to KUDU-1034 fix, the client continued to retry the operation to the same tablet server again and again, not invalidating the entry in its meta-cache. After KUDU-1034 fix, the client started marking the tserver as failed and switching to another one, calling GetTableLocations() after every failure. In the test scenario, the master was not responding fast enough to sustain the rate of adding new entries into the client- and the master-side queues, so eventually the client timed out on the GetTableLocations() calls. As a result, the expected tablet crash hadn't happened because there were too few write operations coming to the tserver to trigger the crash. Having short write timeout is not essential for the test. Bumping the write operation timeout from 100 to 1000 ms allows for the majority of write operations to succeed even in TSAN/ASAN builds. The issue the build-up of GetTableLocations() requests in the client- and master-side queues will be addressed separately, with a separate integration test to cover the specific issue. Change-Id: I6c5449dc9b47062ea9389b25a1b9d906d9de64d9 --- M src/kudu/integration-tests/ts_recovery-itest.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/38/7138/1 -- To view, visit http://gerrit.cloudera.org:8080/7138 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6c5449dc9b47062ea9389b25a1b9d906d9de64d9 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]>
