Tamas Mate 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 10: Code-Review+2

(1 comment)

Thanks for fixing this Steve!
LGTM!

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'm a bit confused on this one.
Sorry, I confused myself, for some reason I thought that Impala supports 
executing UDAFs. Now I see that we won't instantiate any UDAF class, so UDAF 
dependencies like 'UDAFEvaluatorResolver' and 'DefaultUDAFEvaluatorResolver' 
won't be accessed



--
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: 10
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: Thu, 08 Sep 2022 07:46:16 +0000
Gerrit-HasComments: Yes

Reply via email to