Greg Rahn has posted comments on this change. ( http://gerrit.cloudera.org:8080/8768 )
Change subject: IMPALA-5317: [DOCS] Doc for DATE_TRUNC() function ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/8768/2/docs/topics/impala_datetime_functions.xml File docs/topics/impala_datetime_functions.xml: http://gerrit.cloudera.org:8080/#/c/8768/2/docs/topics/impala_datetime_functions.xml@400 PS2, Line 400: Strips off fields from a <codeph>TIMESTAMP</codeph> value Can we use "truncates" not "strips off"? eg. truncates to the specified precision http://gerrit.cloudera.org:8080/#/c/8768/2/docs/topics/impala_datetime_functions.xml@455 PS2, Line 455: select now(), date_trunc('microseconds', now()); : +-------------------------------+-----------------------------------+ : | now() | date_trunc('microseconds', now()) | : +-------------------------------+-----------------------------------+ : | 2017-12-05 13:58:04.565403000 | 2017-12-05 13:58:04.565403000 | : +-------------------------------+-----------------------------------+ now() never returns any nanosecond values so probably not a great example -- 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: 2 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: Mon, 11 Dec 2017 04:08:47 +0000 Gerrit-HasComments: Yes
