rahil-c commented on code in PR #5786:
URL: https://github.com/apache/hudi/pull/5786#discussion_r919268069


##########
hudi-utilities/pom.xml:
##########
@@ -233,6 +242,17 @@
       </exclusions>
     </dependency>
 
+    <dependency>
+      <groupId>org.apache.spark</groupId>
+      <artifactId>spark-hive_${scala.binary.version}</artifactId>
+      <exclusions>
+        <exclusion>
+          <groupId>*</groupId>
+          <artifactId>*</artifactId>
+        </exclusion>
+      </exclusions>
+    </dependency>

Review Comment:
   Removing this dependency seems to cause failures within the azure ci 
https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=9855&view=logs&j=dcedfe73-9485-5cc5-817a-73b61fc5dcb0&t=746585d8-b50a-55c3-26c5-517d93af9934
   
   For now opting to keep this dependency. 
   ```
   [ERROR] testBuildHiveSyncConfig{boolean}[1]  Time elapsed: 0.017 s  <<< 
ERROR!
   java.lang.NoClassDefFoundError: org/apache/spark/sql/hive/HiveExternalCatalog
        at 
org.apache.hudi.DataSourceUtils.buildHiveSyncConfig(DataSourceUtils.java:322)
        at 
org.apache.hudi.TestDataSourceUtils.testBuildHiveSyncConfig(TestDataSourceUtils.java:261)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
   
   ``` 



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to