apache / apache/incubator-xtable
The Hudi example at https://xtable.apache.org/docs/hms#register-the-target-table-in-hive-metastore is not the command for HMS sync.
- Dominant language
- Java
- Stars
- 1.2k
- Forks
- 212
- Avg merge
- 4d 9h
- Merged PRs (30d)
- 16
Description
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/incubator-xtable/issues?q=is%3Aissue) 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?
- [X] I am willing to submit a PR!
- [ ] I am willing to submit a PR but need help getting started!
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
Contributor guide
No contributing guide indexed for this repository
Research direction
Open the “Register the target table in Hive Metastore” section at https://xtable.apache.org/docs/hms#register-the-target-table-in-hive-metastore and compare its example with the hudi-hive-sync documentation. Update the example to use the HMS sync command described in the issue, including the metastore URI and --sync-mode hms. Done means the page shows the correct HMS sync command.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- shell
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100