sandeep akinapelli has posted comments on this change. ( http://gerrit.cloudera.org:8080/8768 )
Change subject: IMPALA-5317: [DOCS] Doc for DATE_TRUNC() function ...................................................................... Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/8768/4/docs/topics/impala_datetime_functions.xml File docs/topics/impala_datetime_functions.xml: http://gerrit.cloudera.org:8080/#/c/8768/4/docs/topics/impala_datetime_functions.xml@403 PS4, Line 403: values TIMESTAMP value or values? http://gerrit.cloudera.org:8080/#/c/8768/4/docs/topics/impala_datetime_functions.xml@422 PS4, Line 422: TS caps on TS and small TS mentioned earlier?? http://gerrit.cloudera.org:8080/#/c/8768/4/docs/topics/impala_datetime_functions.xml@430 PS4, Line 430: one secon what do you think about "one second" vs "second" -- To view, visit http://gerrit.cloudera.org:8080/8768 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifcf38903bb10db12cbb8d73a2dc875aef29cd359 Gerrit-Change-Number: 8768 Gerrit-PatchSet: 4 Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Greg Rahn <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: sandeep akinapelli <[email protected]> Gerrit-Comment-Date: Tue, 12 Dec 2017 06:46:19 +0000 Gerrit-HasComments: Yes
