Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18927 )
Change subject: IMPALA-11528: Catalogd should start up with a corrupt Hive function. ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/18927/9/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java File fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java: http://gerrit.cloudera.org:8080/#/c/18927/9/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java@1767 PS9, Line 1767: } catch (Exception e) { : LOG.error("Skipping function load: " + function.getFunctionName(), e); : } : } > nit: could be multi catch parameter: Done http://gerrit.cloudera.org:8080/#/c/18927/2/java/shaded-deps/hive-exec/pom.xml File java/shaded-deps/hive-exec/pom.xml: http://gerrit.cloudera.org:8080/#/c/18927/2/java/shaded-deps/hive-exec/pom.xml@103 PS2, Line 103: UDAF.class > I think UDAFEvaluatorResolver won't be included, line 111 points to 'ql/udf I'm a bit confused on this one. So I looked in FunctionUtils.java on the Hive side here: https://github.com/apache/hive/blob/release-3.1.2-rc0/ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionUtils.java#L93-L108 Am I looking at the wrong code? Can you point me to the code where you see this class being used? http://gerrit.cloudera.org:8080/#/c/18927/9/java/test-corrupt-hive-udfs/pom.xml File java/test-corrupt-hive-udfs/pom.xml: http://gerrit.cloudera.org:8080/#/c/18927/9/java/test-corrupt-hive-udfs/pom.xml@1 PS9, Line 1: <!-- > nit: <?xml version="1.0" encoding="UTF-8"?> Done -- To view, visit http://gerrit.cloudera.org:8080/18927 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I54e7a1df6d018ba6cf5ecf32dc9946edf86e2112 Gerrit-Change-Number: 18927 Gerrit-PatchSet: 2 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]> Gerrit-Comment-Date: Wed, 07 Sep 2022 14:58:58 +0000 Gerrit-HasComments: Yes
