alexeykudinkin commented on code in PR #6227:
URL: https://github.com/apache/hudi/pull/6227#discussion_r1005995625
##########
pom.xml:
##########
@@ -1728,6 +1736,9 @@
<hudi.spark.common.module>hudi-spark3-common</hudi.spark.common.module>
<scalatest.version>${scalatest.spark3.version}</scalatest.version>
<kafka.version>${kafka.spark3.version}</kafka.version>
+ <parquet.version>1.10.1</parquet.version>
Review Comment:
Good call. This is an invariant though for all profiles, so we need to call
it out holistically
##########
packaging/hudi-flink-bundle/pom.xml:
##########
@@ -639,7 +639,6 @@
<groupId>org.apache.orc</groupId>
<artifactId>orc-core</artifactId>
<version>${orc.version}</version>
- <classifier>nohive</classifier>
Review Comment:
We can't have it in a separate PR, since our code-change is making it
incompatible w/ previous "nohive" version.
And we can't shade Orc or Parquet or any other format which is interfacing
w/ the execution engine b/c we share the classes w/ it (meaning that we pass
these classes' objects from Hudi to the engine, and back)
--
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]
