Hello Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19852
to look at the new patch set (#2).
Change subject: IMPALA-12121: Add non empty check for UDF location in
getLastModifiedTime method
......................................................................
IMPALA-12121: Add non empty check for UDF location in getLastModifiedTime method
getLastModifiedTime() of Function.java assumes that if the UDF is not a BUILTIN
and if the location is not null, it is a non-empty string. However, we may have
UDFs with an empty location if they are in the install location and loaded at
init time.
This patch introduces an isBuiltinOrJava() method that checks if the function is
BUILTIN, JAVA function, or if the location is blank.
Change-Id: Ia6035d1a21e7222f0e95f984a0f4022ba2e89d9f
---
M fe/src/main/java/org/apache/impala/catalog/Function.java
1 file changed, 12 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/52/19852/2
--
To view, visit http://gerrit.cloudera.org:8080/19852
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia6035d1a21e7222f0e95f984a0f4022ba2e89d9f
Gerrit-Change-Number: 19852
Gerrit-PatchSet: 2
Gerrit-Owner: Soumyakanti Das <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Soumyakanti Das <[email protected]>
