Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3504: current timestamp in UTC, i.e. utc_timestamp() ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3137/1/be/src/exprs/timestamp-functions-ir.cc File be/src/exprs/timestamp-functions-ir.cc: Line 826: std::string result = to_simple_string(todayUtc); > It seems inefficient to convert the time into a string in the wrong format Yeah I don't think this is the right approach. We probably want to have something like TimestampValue::UtcToLocal() but in the opposite direction. http://gerrit.cloudera.org:8080/#/c/3137/1/common/function-registry/impala_functions.py File common/function-registry/impala_functions.py: Line 225: STRING A function to get the current timestamp (UTC or not) should return a TIMESTAMP -- To view, visit http://gerrit.cloudera.org:8080/3137 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9bfdb916aeb3a572c66d9dd786e3ea8342030ff0 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Youwei Wang <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-HasComments: Yes
