impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pranay Singh (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected
Date Wed, 23 Aug 2017 22:44:36 GMT
Pranay Singh has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/7791

Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected
......................................................................

IMPALA-5108: idle_session_timeout kicks in later than expected

The issue was caused because sessions could get closed
some time after their timeout expires, because the session
expiration thread only woke up every session-timeout/2
seconds. This fix changes the logic, now the session expiration
thread will wakeup every 1 sec to check whether sessions can
be expired.

The test changes in session-expiry-test is based on introducing
variable names for the numeric constants and to reduce the time
of max_idle_timeout_ms

Change-Id: I72625dcc72764c1b89eb4ecb8725c51c9ca7835b
---
M be/src/service/impala-server.cc
M be/src/service/session-expiry-test.cc
2 files changed, 11 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/91/7791/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7791
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I72625dcc72764c1b89eb4ecb8725c51c9ca7835b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Pranay Singh

Mime
View raw message