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