Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11170 )
Change subject: IMPALA-6481: [DOCS] Documented WIDTH_BUCKET function ...................................................................... Patch Set 3: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/11170/3/docs/topics/impala_math_functions.xml File docs/topics/impala_math_functions.xml: http://gerrit.cloudera.org:8080/#/c/11170/3/docs/topics/impala_math_functions.xml@1679 PS3, Line 1679: if <codeph>expr</codeph> is a : <codeph>NULL</codeph> It really returns NULL if any argument is null: https://github.com/apache/impala/blob/df3f165/be/src/exprs/math-functions-ir.cc#L591 -- To view, visit http://gerrit.cloudera.org:8080/11170 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ife9577a65fe342fde160c7cb5fa666e407d5b093 Gerrit-Change-Number: 11170 Gerrit-PatchSet: 3 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: anujphadke <[email protected]> Gerrit-Comment-Date: Mon, 13 Aug 2018 23:11:27 +0000 Gerrit-HasComments: Yes
