Lars Volker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10893 )

Change subject: IMPALA-6677: [DOCS] Document the next_day function
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/10893/1/docs/topics/impala_datetime_functions.xml
File docs/topics/impala_datetime_functions.xml:

http://gerrit.cloudera.org:8080/#/c/10893/1/docs/topics/impala_datetime_functions.xml@1941
PS1, Line 1941: 00:00:00' the f
> The previous example above uses lower case and doesn't have a space before
This one is still open.


http://gerrit.cloudera.org:8080/#/c/10893/1/docs/topics/impala_datetime_functions.xml@1941
PS1, Line 1941:
> missing word? Also can you wrap the lines at 90 chars within the codeblock?
This one is still open, too.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2dacc86ff69a1016b1863d9db66dd29fd832b715
Gerrit-Change-Number: 10893
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Comment-Date: Fri, 13 Jul 2018 02:07:11 +0000
Gerrit-HasComments: Yes

Reply via email to