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>

Reply via email to