[impala] branch master updated: IMPALA-9757: Bump disconnected_session_timeout in start-impala-cluster.py

[st...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

stakiar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new a7866a9  IMPALA-9757: Bump disconnected_session_timeout in start-impala-cluster.py
a7866a9 is described below

commit a7866a94578be6289bbac31686de4d9032ad9261
Author: Sahil Takiar <ta...@gmail.com>
AuthorDate: Mon Jun 1 09:22:14 2020 -0700

    IMPALA-9757: Bump disconnected_session_timeout in start-impala-cluster.py
    
    Bump the disconnected_session_timeout to 6 hours in
    ./bin/start-impala-cluster.py.
    
    This reduces test flakiness when running tests against the mini-cluster
    using the hs2-http protocol. The issue is that a lot of the E2E tests
    open a hs2-http connection on test startup, but might not use the
    connection for a long time. The connection gets cleaned up and then
    tests start to fail with "HiveServer2Error: Invalid session id"
    exceptions.
    
    The commonly happens in exhaustive tests where we add test dimensions on
    the protocol used to execute E2E tests. This causes the test to switch
    between the beeswax, hs2, and hs2-http protocols. If a test spends over
    an hour using the beeswax protocol, the hs2-http will get closed.
    
    Change-Id: I061a6f96311d406daee14454f71699c8727292d1
    Reviewed-on: http://gerrit.cloudera.org:8080/16014
    Reviewed-by: Tim Armstrong <ta...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 bin/start-impala-cluster.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/start-impala-cluster.py b/bin/start-impala-cluster.py
index 4f1269a..bc38149 100755
--- a/bin/start-impala-cluster.py
+++ b/bin/start-impala-cluster.py
@@ -151,7 +151,7 @@ KUDU_RPC_TIMEOUT = build_flavor_timeout(0, slow_build_timeout=60000)
 
 # HTTP connections don't keep alive their associated sessions. We increase the timeout
 # during builds to make spurious session expiration less likely.
-DISCONNECTED_SESSION_TIMEOUT = 3600
+DISCONNECTED_SESSION_TIMEOUT = 60 * 60 * 6
 
 def check_process_exists(binary, attempts=1):
   """Checks if a process exists given the binary name. The `attempts` count allows us to