Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10208 )
Change subject: IMPALA-6866: Rework timeouts for test_exchange_delays.py ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/10208/1/tests/custom_cluster/test_exchange_delays.py File tests/custom_cluster/test_exchange_delays.py: http://gerrit.cloudera.org:8080/#/c/10208/1/tests/custom_cluster/test_exchange_delays.py@a29 PS1, Line 29: > Removing these two means test_exchange_small_delay and test_exchange_large_ This is intentional. The symptom that caused us to fix IMPALA-6811 had to do with the zero rows case. The other tests were working fine with their existing timeouts. Splitting this out allows the zero rows case to get the timeout it needs and these can go back to what they were. -- To view, visit http://gerrit.cloudera.org:8080/10208 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2e919a4e502b1e6a4156aafbbe4b5ddfe679ed89 Gerrit-Change-Number: 10208 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Comment-Date: Thu, 03 May 2018 18:58:39 +0000 Gerrit-HasComments: Yes
