codope commented on pull request #4551: URL: https://github.com/apache/hudi/pull/4551#issuecomment-1013594138
> @codope i had to revert these changes in my PR, since Presto queries are failing after rebase: > > ``` > 2022-01-14T20:45:04.265Z WARN hive-hive-0 com.facebook.presto.hive.util.ResumableTasks ResumableTask completed exceptionally > java.lang.NoClassDefFoundError: org/apache/avro/message/BinaryMessageEncoder > at org.apache.hudi.avro.model.HoodieMetadataRecord.<clinit>(HoodieMetadataRecord.java:23) @alexeykudinkin Let's not revert this. Instead, we should upgrade the presto version in hudi integ test. Currently, it is 0.217, over 3 years old which did not package avro.message. We want our bundles to be as lightweight as possible and so rely on deps provided by presto as much as possible. Moreover, 0.217 is far removed from the reality. It does not contain the hudi-specific changes that we did in Presto. Also, most Hudi users that I have interacted with are on 0.246 or later. -- 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-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
