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 extra_properties to hive table properties #16592

Closed

Conversation

Praveen2112
Copy link
Member

Description

This PR allows us to pass through additional properties to Hive when creating table in Trino.
The additional properties can provided in the following format

extra_properties = map_from_entries(ARRAY[ROW('extra.property.one', 'one'),ROW('extra.property.two', 'two')]

Additional context and related issues

Fixes #954
Extension of #9475

Release notes

(x) Release notes are required, with the following suggested text:

# Section
* Introduce `extra_properties` for adding arbitrary properties to hive table. ({issue}`954`)

Will be doing a self review before making it ready for review.

@cla-bot cla-bot bot added the cla-signed label Mar 16, 2023
@Praveen2112 Praveen2112 force-pushed the hive-extra-table-properties branch from ec832b5 to 2c1a713 Compare March 16, 2023 14:24
@github-actions github-actions bot added hive Hive connector tests:hive labels Mar 16, 2023
@Praveen2112 Praveen2112 force-pushed the hive-extra-table-properties branch 3 times, most recently from 2c0fc6f to 13e16cd Compare March 21, 2023 07:05
Praveen2112 and others added 2 commits March 27, 2023 11:44
- Remove unused methods
- Extract rewriteProperty to consume `Function<String, I>`
@Praveen2112 Praveen2112 force-pushed the hive-extra-table-properties branch 4 times, most recently from e4d087e to 3d148c3 Compare March 31, 2023 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector
Development

Successfully merging this pull request may close these issues.

Add Hive table property to for arbitrary properties
2 participants