[ https://issues.apache.org/jira/browse/IMPALA-9548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17067340#comment-17067340 ]
ASF subversion and git services commented on IMPALA-9548: --------------------------------------------------------- Commit 46a659871fa349f6aa828d7ea39fe19f64a3ab27 in impala's branch refs/heads/master from Sahil Takiar [ https://gitbox.apache.org/repos/asf?p=impala.git;h=46a6598 ] IMPALA-9548: UdfExecutorTest failures after HIVE-22893 HIVE-22893: "Enhance data size estimation for fields computed by UDFs" modified o.a.h.hive.ql.udf.UDFSubstr and added a dependency on a few new Hive classes located in the hive-exec jar. These classes include: o.a.h.hive.ql.plan.ColStatistics o.a.h.hive.ql.stats.estimator.StatEstimator o.a.h.hive.ql.stats.estimator.StatEstimatorProvider The test UdfExecutorTest#HiveStringsTest loads the class UDFSubstr and thus needs to load the aforementioned stats classes as well. shaded-deps/pom.xml selectively pulls in certain classes from the hive-exec jar, and excludes all others. This patch simply addes the necessary stats classes to load UDFSubstr. Thus, fixing UdfExecutorTest. Testing: * Ran core tests with CDP_BUILD_NUMBER=2244454 and USE_CDP_HIVE=true, validated that UdfExecutorTest now passes Change-Id: I5b3d5df08c2d48d21293d5a5308eb453f40184bf Reviewed-on: http://gerrit.cloudera.org:8080/15544 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> > UdfExecutorTest failures after HIVE-22893 > ----------------------------------------- > > Key: IMPALA-9548 > URL: https://issues.apache.org/jira/browse/IMPALA-9548 > Project: IMPALA > Issue Type: Bug > Reporter: Sahil Takiar > Assignee: Sahil Takiar > Priority: Major > > HIVE-22893 added a dependency on StatEstimatorProvider to certain UDFs. This > causes {{UdfExecutorTest}} to start failing. {{shaded-deps/pom.xml}} defines > a specific set of classes that a pulled in from the {{hive-exec}} jar, > {{StatEstimatorProvider}} just needs to be added to that list. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org For additional commands, e-mail: issues-all-h...@impala.apache.org