Saurabh Katiyal 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:
> Patch Set 9: Code-Review+1
>
> (2 comments)
It is made more generic that why i avoided to put warning in the name
For example ,
If in future we want to add additional information in log we can use like below:
log_timestamp("Info", "some information we want to add with timestamp")
Impala shell stdout would be like :
++++++++++
2024-06-18 22:12:44 [Info] some information we want to add with timestamp
++++++++++
--
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 <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Saurabh Katiyal <[email protected]>
Gerrit-Comment-Date: Tue, 18 Jun 2024 14:04:44 +0000
Gerrit-HasComments: No
