Alex Behm 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") > That doesn't avoid caching of constant expression values in the backend tho Yes, I see your point, but that seems to be an existing bug, so does not have the same urgency as fixing the uuid() regression specifically. It's definitely good to fix as follow-on work. -- 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