Hello Daniel Becker, Joe McDonnell, Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/18207 to look at the new patch set (#7). Change subject: IMPALA-11109: Catch class loading error for UDFs. ...................................................................... IMPALA-11109: Catch class loading error for UDFs. Fixes regression caused by IMPALA-10997. If a UDF fails to load, an exception needs to be caught and logged or else catalogd will not start up. The method HiveLegacyJavaFunction.extract() will catch any hidden exception thrown, and rethrow it as a CatalogException. The caller already handles the CatalogException and logs the message appropriately. The specific "hidden" exception thrown in our regression tests was a "ClassNotFoundException", but in case there are other exceptions that are thrown, we catch the generic "Throwable" exception because the "extract" method should not prevent the server from coming up. Change-Id: I16813209cd4c2367c45e569c2aac13eff7ce2dbf --- M fe/src/main/java/org/apache/impala/hive/executor/HiveLegacyJavaFunction.java M fe/src/test/java/org/apache/impala/hive/executor/HiveLegacyJavaFunctionTest.java 2 files changed, 23 insertions(+), 14 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/07/18207/7 -- To view, visit http://gerrit.cloudera.org:8080/18207 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I16813209cd4c2367c45e569c2aac13eff7ce2dbf Gerrit-Change-Number: 18207 Gerrit-PatchSet: 7 Gerrit-Owner: Steve Carlin <scar...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Steve Carlin <scar...@cloudera.com>