Laurel Hale has posted comments on this change.

Change subject: IMPALA-4107 [DOCS] APPX_MEDIAN cuts string to 10 chars
......................................................................


Patch Set 3:

(2 comments)

> Thanks a lot for change. It's ok for me.

Thanks, Peter!

http://gerrit.cloudera.org:8080/#/c/7094/2/docs/topics/impala_appx_median.xml
File docs/topics/impala_appx_median.xml:

Line 76:     
> remove whitespace
I think this white space is an artifact of Gerrit. It doesn't appear in the 
source XML.


Line 78:       The <codeph>APPX_MEDIAN</codeph> function returns only the first 
10 characters for
> string,varchar,char
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I141787452e74ecdb491765cd7fd4c9a771c5bbc2
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Laurel Hale <lau...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Laurel Hale <lau...@cloudera.com>
Gerrit-Reviewer: Peter Brejcak
Gerrit-HasComments: Yes

Reply via email to