Re: [PR] [SPARK-46305][BUILD][SS] Remove the special Zookeeper version in the `streaming-kafka-0-10` and `sql-kafka-0-10` modules [spark]

["LuciferYang (via GitHub)" <gi...@apache.org>]

LuciferYang commented on code in PR #44230:
URL: https://github.com/apache/spark/pull/44230#discussion_r1418824758


##########
connector/kafka-0-10-sql/pom.xml:
##########
@@ -122,11 +122,9 @@
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-minikdc</artifactId>
     </dependency>
-    <!-- Kafka embedded server uses Zookeeper 3.5.7 API -->

Review Comment:
   cc @dongjoon-hyun 
   
   Currently, Spark uses Kafka 3.4.1, and its embedded server uses Zookeeper 3.6.4. If we wish to continue using a special version of Zookeeper in these two modules, please let me know.
   
   



-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org