Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17551 )

Change subject: IMPALA-10729: [DOCS] Document SHA1 and SHA2 functions
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/17551/1/docs/topics/impala_functions.xml
File docs/topics/impala_functions.xml:

http://gerrit.cloudera.org:8080/#/c/17551/1/docs/topics/impala_functions.xml@70
PS1, Line 70:
Unnecessary indentation here. I cleaned it up.


http://gerrit.cloudera.org:8080/#/c/17551/3/docs/topics/impala_functions.xml
File docs/topics/impala_functions.xml:

http://gerrit.cloudera.org:8080/#/c/17551/3/docs/topics/impala_functions.xml@100
PS3, Line 100:     <table id="impala_functions" frame="none" colsep="0" 
rowsep="0">
It looks like we normally also add links to all functions in this table.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I379de6075af2168715e71edfec26265b6ff5eab7
Gerrit-Change-Number: 17551
Gerrit-PatchSet: 3
Gerrit-Owner: Shajini Thayasingh <[email protected]>
Gerrit-Reviewer: Amogh Margoor <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Mon, 29 Apr 2024 19:39:48 +0000
Gerrit-HasComments: Yes

Reply via email to