Peter Rozsa has posted comments on this change. ( http://gerrit.cloudera.org:8080/21735 )
Change subject: IMPALA-7086: Cache timezone in *_utc_timestamp() ...................................................................... Patch Set 7: Code-Review+1 (2 comments) Nice patch Mihaly! http://gerrit.cloudera.org:8080/#/c/21735/7//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/21735/7//COMMIT_MSG@10 PS7, Line 10: pretty consistent time nit: it's more concise without pretty http://gerrit.cloudera.org:8080/#/c/21735/7/be/src/exprs/timestamp-functions.cc File be/src/exprs/timestamp-functions.cc: http://gerrit.cloudera.org:8080/#/c/21735/7/be/src/exprs/timestamp-functions.cc@112 PS7, Line 112: stringstream ss; nit: if it's refactored to a separate function, stringstream could be replaced with Substitute similarly to L158 -- To view, visit http://gerrit.cloudera.org:8080/21735 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icdf5ff82c5d0554333aef1bc3bba034a4cf48230 Gerrit-Change-Number: 21735 Gerrit-PatchSet: 7 Gerrit-Owner: Mihaly Szjatinya <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Mihaly Szjatinya <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Comment-Date: Thu, 05 Sep 2024 14:04:37 +0000 Gerrit-HasComments: Yes
