[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 2: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 2
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 06:57:23 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..

IMPALA-7943: Bump the default client timeout set on impala-shell

As part of IMPALA-7555, we added a default socket timeout of 5 seconds
when connecting to an impalad. Under heavy load with kerberos and SSL
enabled, we could hit this default timeout. This change bumps up the
timeout to 60 secs to make the impala-shell more robust.

Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Reviewed-on: http://gerrit.cloudera.org:8080/12051
Reviewed-by: Impala Public Jenkins 
Reviewed-by: Lars Volker 
Tested-by: Impala Public Jenkins 
---
M shell/impala_client.py
M shell/impala_shell_config_defaults.py
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified
  Lars Volker: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 3
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 1:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/1556/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 1
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 03:29:31 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Anuj Phadke (Code Review)
Anuj Phadke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 1:

1. opened a listening socket in one terminal
#anuj@anuj-OptiPlex-9020:~/Impala$ netcat -l 12345

2. Ran the shell in an another window -
anuj@anuj-OptiPlex-9020:~/Impala$ time ./bin/impala-shell.sh --ssl -i 
localhost:12345 -q "show tables"
Starting Impala Shell without Kerberos authentication
SSL is enabled. Impala server certificates will NOT be verified (set --ca_cert 
to change)
/home/anuj/Impala/toolchain/thrift-0.9.3-p4/python/lib/python2.7/site-packages/thrift/transport/TSSLSocket.py:80:
 DeprecationWarning: 3th positional argument is deprecated. Use keyward 
argument insteand.
  DeprecationWarning)
/home/anuj/Impala/toolchain/thrift-0.9.3-p4/python/lib/python2.7/site-packages/thrift/transport/TSSLSocket.py:80:
 DeprecationWarning: 4th positional argument is deprecated. Use keyward 
argument insteand.
  DeprecationWarning)
/home/anuj/Impala/toolchain/thrift-0.9.3-p4/python/lib/python2.7/site-packages/thrift/transport/TSSLSocket.py:80:
 DeprecationWarning: 5th positional argument is deprecated. Use keyward 
argument insteand.
  DeprecationWarning)
/home/anuj/Impala/toolchain/thrift-0.9.3-p4/python/lib/python2.7/site-packages/thrift/transport/TSSLSocket.py:216:
 DeprecationWarning: validate is deprecated. Use cert_reqs=ssl.CERT_NONE instead
  DeprecationWarning)
No handlers could be found for logger "thrift.transport.TSSLSocket"
Error connecting: TTransportException, Could not connect to localhost:12345: 
_ssl.c:495: The handshake operation timed out
Not connected to Impala, could not execute queries.

real1m0.228s
user0m0.132s
sys 0m0.040s


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 1
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 03:01:44 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 2: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 2
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 03:04:32 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 2: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 2
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 03:02:04 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 2:

Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/3536/ 
DRY_RUN=false


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 2
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Lars Volker 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 03:02:05 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12051 )

Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..


Patch Set 1: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/12051
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 1
Gerrit-Owner: Anuj Phadke 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Comment-Date: Fri, 07 Dec 2018 02:58:19 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7943: Bump the default client timeout set on impala-shell

2018-12-06 Thread Anuj Phadke (Code Review)
Anuj Phadke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12051


Change subject: IMPALA-7943: Bump the default client timeout set on impala-shell
..

IMPALA-7943: Bump the default client timeout set on impala-shell

As part of IMPALA-7555, we added a default socket timeout of 5 seconds
when connecting to an impalad. Under heavy load with kerberos and SSL
enabled, we could hit this default timeout. This change bumps up the
timeout to 60 secs to make the impala-shell more robust.

Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
---
M shell/impala_client.py
M shell/impala_shell_config_defaults.py
2 files changed, 2 insertions(+), 2 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifc40069e86cbf93634320804efba003fb5551afe
Gerrit-Change-Number: 12051
Gerrit-PatchSet: 1
Gerrit-Owner: Anuj Phadke