[GitHub] [hudi] hackergin commented on a diff in pull request #6628: [HUDI-4806] Use Avro version from the root pom for Flink bundle

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

hackergin commented on code in PR #6628:
URL: https://github.com/apache/hudi/pull/6628#discussion_r971586922


##########
packaging/hudi-flink-bundle/pom.xml:
##########
@@ -501,8 +501,7 @@
     <dependency>
       <groupId>org.apache.avro</groupId>
       <artifactId>avro</artifactId>
-      <!-- Override the version to be same with Flink avro -->
-      <version>1.10.0</version>
+      <version>${avro.version}</version>

Review Comment:
   > Yes, we've tested with Flink streamer loading data from Kafka datasource in Hudi format. And it works fine
   Hi, @CTTY  I met a java.lang.ClassNotFoundException when using the latest master code.   Class org.apache.avro.LogicalTypes$LocalTimestampMillis  seems to only appear in avro 1.10 version. Please help to confirm this problem, correct me if I am wrong . 
   ```
   Caused by: java.lang.ClassNotFoundException: org.apache.hudi.org.apache.avro.LogicalTypes$LocalTimestampMillis
           at java.net.URLClassLoader.findClass(URLClassLoader.java:382) ~[?:1.8.0_202]
           at java.lang.ClassLoader.loadClass(ClassLoader.java:424) ~[?:1.8.0_202]
           at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349) ~[?:1.8.0_202]
           at java.lang.ClassLoader.loadClass(ClassLoader.java:357) ~[?:1.8.0_202]
           at org.apache.hudi.table.HoodieTableFactory.inferAvroSchema(HoodieTableFactory.java:346) ~[hudi-flink1.14-bundle-0.13.0-SNAPSHOT.jar:0.13.0-SNAPSHOT]
   ```



-- 
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: commits-unsubscribe@hudi.apache.org

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