Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/14201 )
Change subject: IMPALA-8932: shell shouldn't retry kerberos over http ...................................................................... Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/14201/1/shell/impala_client.py File shell/impala_client.py: http://gerrit.cloudera.org:8080/#/c/14201/1/shell/impala_client.py@379 PS1, Line 379: # TODO: if we enable this, need to re-enable the automatic kerberos retry logic nit: This comment reads somewhat of. Should it say "if we want to enable this"? Maybe I'm just missing something. -- To view, visit http://gerrit.cloudera.org:8080/14201 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5dde277a6a0ddbe5a919bcf376bbc19f0b48e95e Gerrit-Change-Number: 14201 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Comment-Date: Mon, 09 Sep 2019 23:57:12 +0000 Gerrit-HasComments: Yes
