Tim Armstrong has posted comments on this change. Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5324/1/fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java File fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java: Line 246: if (fnName.equalsIgnoreCase("rand") || fnName.equalsIgnoreCase("random") > Yes, I see your point, but that seems to be an existing bug, so does not ha There is a regression in the 2.8.0 release though because I added more caching of constant values in the backend in this commit: commit 10fa472fa6aa036be02748ae54daed1722449c68 Author: Tim Armstrong <tarmstr...@cloudera.com> Date: Wed Oct 26 10:55:23 2016 -0700 IMPALA-4302,IMPALA-2379: constant expr arg fixes I'll create a JIRA for that since we're not going to fix it as part of this patch, right? -- To view, visit http://gerrit.cloudera.org:8080/5324 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: If2499f5f6ecdcb098623202c8e6dc2d02727194a Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-HasComments: Yes