Tim Armstrong has posted comments on this change.

Change subject: IMPALA-5030: [DOCS] Document nvl2() function
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/7365/1/docs/topics/impala_conditional_functions.xml
File docs/topics/impala_conditional_functions.xml:

PS1, Line 467: if(x is not null
Hmm this is a bit unfortunate but I think just a consequence of how it was 
implemented.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8497e69d0f6db3c5e17b04e49a875c3efb43fab0
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <[email protected]>
Gerrit-Reviewer: Greg Rahn <[email protected]>
Gerrit-Reviewer: John Russell <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Vincent Tran <[email protected]>
Gerrit-HasComments: Yes

Reply via email to