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.

Open
#455 1 comment 0 reactions 0 assignees View on GitHub
bug
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.