Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19852 )
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, or a JAVA function with blank location. Change-Id: Ia6035d1a21e7222f0e95f984a0f4022ba2e89d9f Reviewed-on: http://gerrit.cloudera.org:8080/19852 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/Function.java 1 file changed, 12 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Ia6035d1a21e7222f0e95f984a0f4022ba2e89d9f Gerrit-Change-Number: 19852 Gerrit-PatchSet: 5 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]>
