[GitHub] [hudi] rmpifer commented on a change in pull request #2208: [HUDI-1040] Make Hudi support Spark 3

[GitBox <gi...@apache.org>]

rmpifer commented on a change in pull request #2208:
URL: https://github.com/apache/hudi/pull/2208#discussion_r517159673



##########
File path: packaging/hudi-utilities-bundle/pom.xml
##########
@@ -105,6 +106,7 @@
                   <include>io.prometheus:simpleclient_common</include>
                   <include>com.yammer.metrics:metrics-core</include>
                   <include>org.apache.spark:spark-streaming-kafka-0-10_${scala.binary.version}</include>
+                  <include>org.apache.spark:spark-token-provider-kafka-0-10_${scala.binary.version}</include>

Review comment:
       This was needed when using Kafka + DeltaStreamer in Spark 3.
   ```
   java.lang.NoClassDefFoundError: org/apache/spark/kafka010/KafkaConfigUpdater
       at org.apache.spark.streaming.kafka010.InternalKafkaConsumer.createConsumer(KafkaDataConsumer.scala:115)
   ```
   
   Can we validate this doesn't cause issues when building with Spark 2. I believe this exists only in Spark 3
   
   https://mvnrepository.com/artifact/org.apache.spark/spark-token-provider-kafka-0-10




----------------------------------------------------------------
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:
users@infra.apache.org