Michael Ho has posted comments on this change. Change subject: IMPALA-3018: Don't return NULL on zero length allocations. ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3601/1//COMMIT_MSG Commit Message: Line 19: > could changing this cause trouble for UDFs that assume NULL is returned for IMHO, it's a bit too far fetched to rely on such behavior. We didn't specify the behavior of zero length allocation in the header file so we aren't breaking compatibility strictly speaking. Can you think of a reason which user may rely on such behavior ? -- To view, visit http://gerrit.cloudera.org:8080/3601 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id8c3211f4d9417f44b8018ccc58ae182682693da Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Michael Ho <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-HasComments: Yes
