Attila Jeges has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14896 )

Change subject: IMPALA-9035: Simplify casting string to timestamp. This change 
will help with queries generated by some BI tools. Case 1: Simplify 'string -> 
bigint -> timestamp' TO 'string -> timestamp': cast(unix_timestamp('timestr') 
as timestamp) -> cast('timestr' as
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14896/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/14896/4//COMMIT_MSG@8
PS4, Line 8: This change will help with queries generated by some BI tools.
           : Case 1:
           : Simplify 'string -> bigint -> timestamp' TO 'string -> timestamp':
           : cast(unix_timestamp('timestr') as timestamp) -> cast('timestr' as 
timestamp)
           : Case 2:
           : Simplify 'string[fmt] -> bigint -> timestamp' TO 'string -> 
timestamp':
           : cast(unix_timestamp('timestr', 'fmt') as timestamp) -> 
to_timestamp('timestr', 'fmt')
Also, please wrap long lines at 70 characters.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4ed72d6e7886eaf50d2be60cf45170ffaef5e72d
Gerrit-Change-Number: 14896
Gerrit-PatchSet: 4
Gerrit-Owner: Zhou Xu <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zhou Xu <[email protected]>
Gerrit-Comment-Date: Thu, 09 Jan 2020 10:38:44 +0000
Gerrit-HasComments: Yes

Reply via email to