henrib commented on code in PR #6936:
URL: https://github.com/apache/hive/pull/6936#discussion_r4494592010
##########
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java:
##########
@@ -1991,6 +1991,12 @@ public enum ConfVars {
"hive.metastore.iceberg.catalog.vended-credentials.enabled", false,
"Boolean flag to enable credential vending on Iceberg REST Catalog"
),
+
ICEBERG_CATALOG_CLIENT_SUPPLIED_OWNER_ENABLED("metastore.iceberg.catalog.client-supplied-owner.enabled",
+ "hive.metastore.iceberg.catalog.client-supplied-owner.enabled", false,
+ "Whether the HMS Iceberg REST catalog records the owner a client
requests through the " +
+ "'hive.metastore.table.owner' property when creating a table or
view. When false (the default), the " +
+ "authenticated caller is recorded as owner."
+ ),
Review Comment:
Agreed, no use case I know of needs it: delegated creation is better served
by impersonation. Removed in e16e190205: the REST catalog always records the
authenticated caller as owner; the embedded `HiveCatalog` setting stays as is
for compatibility.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
