Added Support For Configurable Updatable Table Properties per Iceberg Catalog Type #23905
Conversation
| @@ -196,4 +197,9 @@ void createMaterializedView( | |||
| void updateColumnComment(ConnectorSession session, SchemaTableName schemaTableName, ColumnIdentity columnIdentity, Optional<String> comment); | |||
|
|
|||
| Optional<CatalogSchemaTableName> redirectTable(ConnectorSession session, SchemaTableName tableName, String hiveCatalogName); | |||
|
|
|||
| default Set<String> getUpdatableTableProperties() | |||
There was a problem hiding this comment.
Is this an extension point for your fork? Why not do that in your repository if so?
There was a problem hiding this comment.
It was a simple change that could be contributed back to OSS hence opened up for here.
There was a problem hiding this comment.
@ebyhr and then I'll backport this change if this goes through.
but even though I am a member of the trino community and I did the CLA thing, not sure why it says i am not authorized 🤔
There was a problem hiding this comment.
First of all, I am not convinced with this change. We usually deny fork-specific changes.
What do you mean > "i am not authorized"?
|
This pull request has gone a while without any activity. Tagging for triage help: @mosabua |
|
Let me close as a stale PR. |
Description
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.