Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/9800 )
Change subject: IMPALA-6724: Incorrect exception handling in create function statement ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/9800/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/9800/3//COMMIT_MSG@13 PS3, Line 13: Existing behavior: To get ahead of all the comments regarding behavior and compatibility, it might be good to make an exhaustive list of all cases, and then show the old/new behaviors. The 2 dimensions are: * targets _impala_builtins database or not * function name has a built-in with the same name or not So a total of 4 cases, each with old vs. new behavior. -- To view, visit http://gerrit.cloudera.org:8080/9800 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic30df56ac276970116715c14454a5a2477b185fa Gerrit-Change-Number: 9800 Gerrit-PatchSet: 3 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Comment-Date: Mon, 26 Mar 2018 18:20:18 +0000 Gerrit-HasComments: Yes