lvyanquan commented on code in PR #4339:
URL: https://github.com/apache/flink-cdc/pull/4339#discussion_r2988631544


##########
docs/content/docs/connectors/flink-sources/datastream-api-package-guidance.md:
##########
@@ -169,7 +169,7 @@ This guide provides a simple `pom.xml` example for 
packaging DataStream job JARs
                                     
<include>io.debezium:debezium-ddl-parser</include>
                                     
<include>io.debezium:debezium-connector-mysql</include>
                                     
<include>org.apache.flink:flink-connector-debezium</include>
-                                    
<include>org.apache.flink:flink-connector-mysql-cdc</include>
+                                    
<include>org.apache.flink:flink-sql-connector-mysql-cdc</include>

Review Comment:
   This is because when running jobs on `Flink 2.2`, we depend on two 
flink-shaded-guava versions: `31.1-jre-17.0`(as we directly import 
`org.apache.flink.shaded.guava31` in the code) and `33.4.0-jre-20.0`. When 
running in an IDE (not submitting a JAR to the cluster), since they share the 
same groupId and artifactId, only the `33.4.0-jre-20.0` dependency can be 
provided. Therefore, an uber JAR is needed to provide the other 
flink-shaded-guava dependency.
   
   This dependency is only required for `Flink 2.2`. For `Flink 1.20` jobs, the 
dependency configuration remains the same as before.



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