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.

Sign up for GitHub

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

Jump to bottom

Add intelligent tiering storage class #16028

Merged
merged 1 commit into from
May 12, 2021
Merged

Add intelligent tiering storage class #16028

merged 1 commit into from
May 12, 2021

Conversation

v-jizhang
Copy link
Contributor

@v-jizhang v-jizhang commented Apr 30, 2021 •
edited
Loading

Cherry pick of trinodb/trino#3032

Co-authored-by Arpit Solanki [email protected]

Test plan - (Please fill in how you tested your changes)

Please make sure your submission complies with our Development, Formatting, and Commit Message guidelines. Don't forget to follow our attribution guidelines for any code copied from other projects.

Fill in the release notes towards the bottom of the PR description.
See Release Notes Guidelines for details.

== RELEASE NOTES ==

Hive Connector Changes
* Added intelligent tiering storage class.
  The S3 storage class to use when writing the data. STANDARD and INTELLIGENT_TIERING storage classes are supported. Default storage class is STANDARD
:doc:`/connector/hive`

@v-jizhang
Add intelligent tiering storage class
c3d13eb 
Cherry pick of trinodb/trino#3032

Co-authored-by Arpit Solanki <[email protected]>
@aweisberg aweisberg self-requested a review May 12, 2021 19:42
aweisberg
aweisberg approved these changes May 12, 2021
View reviewed changes
Copy link
Contributor

@aweisberg aweisberg left a comment

Choose a reason for hiding this comment

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

Love it!

@aweisberg aweisberg requested review from rschlussel and arhimondr May 12, 2021 19:48
@arhimondr arhimondr merged commit 7e4fe3d into prestodb:master May 12, 2021
@sujay-jain sujay-jain mentioned this pull request May 21, 2021
Merged
10 tasks
@ajaygeorge ajaygeorge mentioned this pull request May 26, 2021
Closed
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aweisberg aweisberg aweisberg approved these changes

@rschlussel rschlussel Awaiting requested review from rschlussel

@arhimondr arhimondr Awaiting requested review from arhimondr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@v-jizhang @aweisberg @arhimondr