Skip to content

Rename hive.s3.storage-class-filter and add documentation#25633

Merged
pettyjamesm merged 1 commit intotrinodb:masterfrom
zhaner08:xuanyu_update_s3_storage_class_filter_config_name
Apr 22, 2025
Merged

Rename hive.s3.storage-class-filter and add documentation#25633
pettyjamesm merged 1 commit intotrinodb:masterfrom
zhaner08:xuanyu_update_s3_storage_class_filter_config_name

Conversation

@zhaner08
Copy link
Copy Markdown
Contributor

@zhaner08 zhaner08 commented Apr 19, 2025

Description

Rename hive.s3.storage-class-filter to hive.s3-glacier-filter and add documentation as a follow up item for PR: #24979

Additional context and related issues

Release notes

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

## Hive
- Rename hive configuration property `hive.s3.storage-class-filter` to `hive.s3-glacier-filter` to better reflect its purpose ({issue}`25633`)

@cla-bot cla-bot bot added the cla-signed label Apr 19, 2025
@github-actions github-actions bot added docs hive Hive connector labels Apr 19, 2025
@zhaner08 zhaner08 requested a review from pettyjamesm April 19, 2025 21:18
@zhaner08 zhaner08 self-assigned this Apr 19, 2025
@zhaner08 zhaner08 removed the request for review from pettyjamesm April 21, 2025 03:28
@zhaner08 zhaner08 force-pushed the xuanyu_update_s3_storage_class_filter_config_name branch from 991cf74 to e6b1f97 Compare April 21, 2025 03:29
Copy link
Copy Markdown
Member

@pettyjamesm pettyjamesm left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for following up @zhaner08

@pettyjamesm pettyjamesm merged commit 775445a into trinodb:master Apr 22, 2025
61 checks passed
@github-actions github-actions bot added this to the 475 milestone Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants