-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for hoodie.metadata.enable in Hudi connector #22289
Comments
xtable developers asked for an additional setting "hoodie.metadata.enable=true" to be set in the Hudi Trino Connector. See apache/incubator-xtable#460 (comment) fro details |
@krvikash and @Praveen2112. I'm new to Trino and I see that you previously committed to HudiConfig. What is the process to add this additional setting? Is it something that I can do? |
Setting |
@ebyhr what do you suggest then since xtable says that setting is needed to read hudi files? |
@ebyhr I'd be happy to test. I'm documenting my testing at apache/incubator-xtable#461 |
Per conversation with @codope, he suggested to use Trino 418. |
related. #18840 (comment) |
Here is what worked for me https://github.com/soumilshah1995/HudiDeltaStreamer-SCD-Trino
|
@soumilshah1995 In your example, you're using Trino 400 which is older than Trino 418 version. That's why yours works. If you try Trino 419 or newer, I believe you'll hit the same issue as I did. |
yes that's why I was using older version of trino |
cc: @codope |
Comment from @yihua Trino 418 depends on Hudi 0.12.3 while Iceberg table with Hudi metadata requires Hudi 0.14.0+ so Trino 418 does not work in your use case. |
Trino version: 428
Used Apache xTable to source Delta Lake data to target / generate Apache Hudi data in S3. Then I registered the Apache Hudi generated data into HMS. Both processes show no errors. See apache/incubator-xtable#460 for details.
The advice from xtable dev is to set
hoodie.metadata.enable=true
in hudi.properties but this setting is defunct.just to try it out, trying also
hudi.metadata-enabled=true
The text was updated successfully, but these errors were encountered: