This is an automated email from the ASF dual-hosted git repository. bharathv pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 7cee01d1ba7ab6018bf6d7e4d90f4758fd458ce0 Author: Andrew Sherman <[email protected]> AuthorDate: Tue Feb 12 16:17:13 2019 -0800 IMPALA-8194: wait longer to detect JVM pause in TestPauseMonitor. The test 'test_jvm_pause_monitor_logs_entries' stops and starts an impalad, and confirms that that the JVM pause monitor detects the pause by looking for a specific message in the log. In a test run the test failed to find the correct message after sleeping for 1.2 seconds. Because the test notes the last message that it sees in the log, we can observe that the test would have found the correct message if it had waited for just a few more milliseconds. This change increases the time that the test waits to 2 seconds. TESTING: Ran end-to-end tests cleanly and checked that test_jvm_pause_monitor_logs_entries ran OK. Change-Id: I735c0c0ecfd3a9099c9cef332c5e79854bec7b8d Reviewed-on: http://gerrit.cloudera.org:8080/12475 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- tests/custom_cluster/test_pause_monitor.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/custom_cluster/test_pause_monitor.py b/tests/custom_cluster/test_pause_monitor.py index 166316f..bf47190 100644 --- a/tests/custom_cluster/test_pause_monitor.py +++ b/tests/custom_cluster/test_pause_monitor.py @@ -34,7 +34,7 @@ class TestPauseMonitor(CustomClusterTestSuite): time.sleep(5) impalad.kill(signal.SIGCONT) # Wait for over a second for the cache metrics to expire. - time.sleep(1.2) + time.sleep(2) # Check that the pause is detected. self.assert_impalad_log_contains('INFO', "Detected pause in JVM or host machine") # Check that the metrics we have for this updated as well
