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

[docs] added run_syn_tool.sh with HMS example and also added more on S3 changes needed for Trino. #458

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alberttwong
Copy link
Contributor

To fix #455

What is the purpose of the pull request

added run_syn_tool.sh with HMS example and also added more on S3 changes needed for Trino.

Brief change log

  • Doc updates

Verify this pull request

This pull request is a trivial rework / code cleanup without any test coverage.

website/docs/hms.md Outdated Show resolved Hide resolved
website/docs/hms.md Outdated Show resolved Hide resolved
website/docs/hms.md Outdated Show resolved Hide resolved
website/docs/hms.md Outdated Show resolved Hide resolved
Update hms.md
@ghost
Copy link

ghost commented Jun 6, 2024

CI report:

Bot commands @xtable-bot supports the following commands:
  • @xtable-bot run azure re-run the last Azure build

Copy link
Contributor

@vinishjail97 vinishjail97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some nit comments, will merge after addressing them.

website/docs/hms.md Show resolved Hide resolved
@alberttwong alberttwong changed the title Update hms.md [docs] added run_syn_tool.sh with HMS example and also added more on S3 changes needed for Trino. Sep 3, 2024
```shell md title="shell"
java -jar xtable-utilities/target/xtable-utilities-0.2.0-SNAPSHOT-bundled.jar --datasetConfig my_config.yaml
```shell in JDBC md title="shell in JDBC"
java -jar xtable-utilities/target/xtable-utilities-0.1.0-SNAPSHOT-bundled.jar --datasetConfig my_config.yaml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We updated the version to 0.2.0-SNAPSHOT in main branch now.

I feel it's better to use the downloads from maven for the demo going forward. WDYT ?
https://xtable.apache.org/releases/downloads/

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