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

Add docs for AWS Glue support in Hudi #23093

Closed
wants to merge 1 commit into from

Conversation

alberttwong
Copy link

adding documentation that AWS Glue support in Hudi.

Description

Additional context and related issues

Release notes

(X) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Aug 21, 2024
@github-actions github-actions bot added the docs label Aug 21, 2024
- Access to a Hive metastore service (HMS).
- Network access from the Trino coordinator to the HMS.
- Access to a Hive metastore service (HMS) or AWS Glue.
- Network access from the Trino coordinator to the HMS or AWS Glue.
Copy link
Member

Choose a reason for hiding this comment

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

We don't run CI for AWS Glue with Hudi connector at this time. I hesitate documenting it in this situation.

Copy link
Author

Choose a reason for hiding this comment

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

I've tested it personally that it works. I don't know if that helps or not.

@alberttwong alberttwong changed the title Update hudi.md adding documentation that AWS Glue support in Hudi. Aug 24, 2024
Copy link

This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua

@github-actions github-actions bot added the stale label Sep 25, 2024
@mosabua mosabua changed the title adding documentation that AWS Glue support in Hudi. Add docs for AWS Glue support in Hudi Sep 25, 2024
@mosabua mosabua force-pushed the patch-2 branch 2 times, most recently from 133f67a to 2b5637f Compare September 25, 2024 20:40
Co-authored-by: Albert T. Wong <atwong@alumni.uci.edu>
@mosabua
Copy link
Member

mosabua commented Sep 25, 2024

I rebased and updated the PR now .. wdyt @ebyhr @yihua .. should we merge this even though we don't have tests for it?

@ebyhr
Copy link
Member

ebyhr commented Sep 25, 2024

@mosabua I don't think we want to merge this PR unless we add tests.

@mosabua
Copy link
Member

mosabua commented Sep 25, 2024

Sounds good .. @alberttwong and @yihua .. can you maybe add some testing with Glue in this PR or a separate PR? In the meantime I will close this PR.

@mosabua mosabua closed this Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants