Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21426 )

Change subject: IMPALA-12216: Print timestamp for impala-shell errors
......................................................................


Patch Set 9:

There are still some places that we need the timestamp. Can we add them as well?

$ git grep -n 'Caught exception' shell
shell/impala_client.py:795:        print('Caught exception {0}, type={1} when 
listing query options. {2}'
shell/impala_client.py:800:        print('Caught exception {0}, type={1} when 
listing query options. {2}'
shell/impala_client.py:1180:          print('Caught exception {0}, type={1} in 
{2}. {3}, retry after {4} secs'
shell/impala_client.py:1184:          print('Caught exception {0}, type={1} in 
{2}. {3}'
shell/impala_client.py:1190:        print('Caught exception {0}, type={1} in 
{2}. {3}'


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4abbd02aa9f61210b0333495bf191e72c22a5944
Gerrit-Change-Number: 21426
Gerrit-PatchSet: 9
Gerrit-Owner: Saurabh Katiyal <saurabhkati...@gmail.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Saurabh Katiyal <saurabhkati...@gmail.com>
Gerrit-Comment-Date: Wed, 19 Jun 2024 04:19:55 +0000
Gerrit-HasComments: No

Reply via email to