John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9014 )
Change subject: IMPALA-2172, IMPALA-6391: [DOCS] Distinguish char_length() from length() ...................................................................... Patch Set 1: Agree with having Greg look it over. If we can +2 it today though, it can make it into the 2.11 docs. Greg won't be back until next week. How about we +2 this review today and take a longer-term action item to fill in more examples for the string functions, so we can turn up any other examples of STRING / CHAR differences? -- To view, visit http://gerrit.cloudera.org:8080/9014 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I18cabfce66351bb890bfbfc26b93466204a82625 Gerrit-Change-Number: 9014 Gerrit-PatchSet: 1 Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Greg Rahn <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Juan Yu <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Fri, 12 Jan 2018 20:23:21 +0000 Gerrit-HasComments: No
