Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14722 )

Change subject: IMPALA-9141: [DOCS] SQL:2016 date time patterns - Milestone 2
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14722/3/docs/topics/impala_conversion_functions.xml
File docs/topics/impala_conversion_functions.xml:

http://gerrit.cloudera.org:8080/#/c/14722/3/docs/topics/impala_conversion_functions.xml@557
PS3, Line 557: <p>The <codeph>AM</codeph>/<codeph>PM</codeph> input
             :                         length has to match the pattern's 
length. For example,
             :                         the input argument of 
<codeph>A.M.</codeph> is not
             :                         accepted for the 
<varname>pattern</varname> of
             :                           <codeph>AM</codeph>.</p>
> I have just noticed that this part is valid for string to datetime path.
It was from your design doc.
I will remove it.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id31e9ab14b56afc2e0bd7332cac299243a16bb07
Gerrit-Change-Number: 14722
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 03 Dec 2019 19:48:06 +0000
Gerrit-HasComments: Yes

Reply via email to