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

Change subject: IMPALA-8704: ISO:SQL:2016 datetime patterns - Milestone 2
......................................................................


Patch Set 10:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14291/7/tests/query_test/test_cast_with_format.py
File tests/query_test/test_cast_with_format.py:

http://gerrit.cloudera.org:8080/#/c/14291/7/tests/query_test/test_cast_with_format.py@782
PS7, Line 782: # Strict separator matching.
             :     result = self.client.execute("select cast('2001-03-02 
03:10:15' as timestamp format"
             :         "'FXYYYY MM-DD HH12:MI:SS')")
             :     assert result.data == ["NULL"]
             :
             :     result = self.client.execute("select cast('2001-03-03 
03:10:15' as timestamp format"
             :         "'FXYYYY-MM-DD HH12::MI:SS')")
             :     assert result.data == ["NULL"]
             :
             :     result = self.client.execute("select cast('2001-03-04    ' 
as timestamp format"
             :         "'FXYYYY-MM-DD ')")
             :     assert result.data == ["NULL"]
             :
             :     # Strict token length matching.
             :     result = self.client.execute("select cast('2001-3-05' as 
timestamp format "
             :         "'FXYYYY-MM-DD')")
             :     assert result.data == ["NULL"]
> L778 is a positive test for FX modifier. Does that cover what you ask for?
Ok, thanks.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I30d2f6656054371476aaa8bd0d51f572b9369855
Gerrit-Change-Number: 14291
Gerrit-PatchSet: 10
Gerrit-Owner: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 18 Oct 2019 12:10:23 +0000
Gerrit-HasComments: Yes

Reply via email to