Remove invalidate metastore cache procedure from iceberg#25787
Merged
pratyakshsharma merged 1 commit intoAug 16, 2025
Conversation
agrawalreetika
approved these changes
Aug 14, 2025
cf82dae to
e05838e
Compare
tdcmeehan
approved these changes
Aug 14, 2025
7 tasks
pratyakshsharma
added a commit
that referenced
this pull request
Apr 28, 2026
## Description Enable metastore cache invalidation procedure for Iceberg connector now that metastore cache is enabled for iceberg connector as part of #26918. The check for this procedure was removed in #25787. ## Motivation and Context Metastore cache invalidation procedure can be enabled for iceberg connector now. ## Impact <!---Describe any public API or user-facing feature change or any performance impact--> ## Test Plan NA ## Contributor checklist - [ ] Please make sure your submission complies with our [contributing guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md), in particular [code style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style) and [commit standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards). - [ ] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced. - [ ] Documented new properties (with its default value), SQL syntax, functions, or other functionality. - [ ] If release notes are required, they follow the [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines). - [ ] Adequate tests were added if applicable. - [ ] CI passed. - [ ] If adding new dependencies, verified they have an [OpenSSF Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or higher (or obtained explicit TSC approval for lower scores). ## Summary by Sourcery New Features: - Conditionally register the metastore cache invalidation procedure for the Iceberg connector based on metastore client configuration. ## Release Notes Please follow [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines) and fill in the release notes below. ``` == RELEASE NOTES == Iceberg Connector Changes * Add metastore cache invalidation procedure for Iceberg connector ```
msmygit
pushed a commit
to msmygit/presto
that referenced
this pull request
Jun 3, 2026
## Description Enable metastore cache invalidation procedure for Iceberg connector now that metastore cache is enabled for iceberg connector as part of prestodb#26918. The check for this procedure was removed in prestodb#25787. ## Motivation and Context Metastore cache invalidation procedure can be enabled for iceberg connector now. ## Impact <!---Describe any public API or user-facing feature change or any performance impact--> ## Test Plan NA ## Contributor checklist - [ ] Please make sure your submission complies with our [contributing guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md), in particular [code style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style) and [commit standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards). - [ ] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced. - [ ] Documented new properties (with its default value), SQL syntax, functions, or other functionality. - [ ] If release notes are required, they follow the [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines). - [ ] Adequate tests were added if applicable. - [ ] CI passed. - [ ] If adding new dependencies, verified they have an [OpenSSF Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or higher (or obtained explicit TSC approval for lower scores). ## Summary by Sourcery New Features: - Conditionally register the metastore cache invalidation procedure for the Iceberg connector based on metastore client configuration. ## Release Notes Please follow [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines) and fill in the release notes below. ``` == RELEASE NOTES == Iceberg Connector Changes * Add metastore cache invalidation procedure for Iceberg connector ```
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Now that metastore cache is no longer supported for iceberg, we do not need to have
system.invalidate_metastore_cacheprocedure for iceberg either.Motivation and Context
Support for metastore cache for iceberg was removed in this PR #24326
Impact
Users no longer have
system.invalidate_metastore_cacheprocedure for iceberg connector.Test Plan
NA
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.