Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/11741 )
Change subject: IMPALA-7697: Fix flakiness in test_resource_limits ...................................................................... Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/11741/1/testdata/workloads/functional-query/queries/QueryTest/query-resource-limits.test File testdata/workloads/functional-query/queries/QueryTest/query-resource-limits.test: http://gerrit.cloudera.org:8080/#/c/11741/1/testdata/workloads/functional-query/queries/QueryTest/query-resource-limits.test@46 PS1, Line 46: select sleep(10000) I like that the simplification addresses more directly what's being tested here. Can we do the same for the other time limit tests here? -- To view, visit http://gerrit.cloudera.org:8080/11741 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2ba7080f62f0af3e16ef6c304463ebf78dec1b0c Gerrit-Change-Number: 11741 Gerrit-PatchSet: 1 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Fri, 19 Oct 2018 22:28:20 +0000 Gerrit-HasComments: Yes
