Skip to content
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

The Hudi example at https://xtable.apache.org/docs/hms#register-the-target-table-in-hive-metastore is not the command for HMS sync. #455

Open
3 of 4 tasks
alberttwong opened this issue Jun 2, 2024 · 1 comment · May be fixed by #458
Labels
bug Something isn't working

Comments

@alberttwong
Copy link
Contributor

alberttwong commented Jun 2, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Please describe the bug 🐞

Per hudi-hive-sync docs, since we're talking about HMS sync, isn't the command something like

./run_sync_tool.sh  \
--metastore-uris 'thrift://hive-metastore:9083' \
--partitioned-by city \
--base-path 's3a://warehouse/people' \
--database hudi_db \
--table people \
--sync-mode hms 

Are you willing to submit PR?

  • I am willing to submit a PR!
  • I am willing to submit a PR but need help getting started!

Code of Conduct

@vinishjail97
Copy link
Contributor

Hudi Hive Sync has three modes - JDBC, HMS and HIVEQL.
https://github.com/apache/hudi/blob/master/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HoodieHiveSyncClient.java#L94

I have added few comments on your PR for docs improvements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants