shardulm94 commented on a change in pull request #31623:
URL: https://github.com/apache/spark/pull/31623#discussion_r581285561
##########
File path:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveQuerySuite.scala
##########
@@ -1224,7 +1224,10 @@ class HiveQuerySuite extends HiveComparisonTest with
SQLTestUtils with BeforeAnd
test("SPARK-33084: Add jar support Ivy URI in SQL") {
val testData = TestHive.getHiveFile("data/files/sample.json").toURI
withTable("t") {
- sql(s"ADD JAR
ivy://org.apache.hive.hcatalog:hive-hcatalog-core:$hiveVersion")
+ // exclude org.pentaho:pentaho-aggdesigner-algorithm as this transitive
dependency does
+ // not exist on mavencentral and hence cannot be found in the test
environment
+ sql(s"ADD JAR
ivy://org.apache.hive.hcatalog:hive-hcatalog-core:$hiveVersion" +
+ "?exclude=org.pentaho:pentaho-aggdesigner-algorithm")
Review comment:
This is a very infamous dependency brought in by calcite. It does not
exist on mavencentral but it does exist on conjars.org. Here is a Stack
overflow asking about the same issue.
https://stackoverflow.com/questions/32587769/dependency-resolution-issue-in-gradle
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]