deniskuzZ commented on code in PR #6218:
URL: https://github.com/apache/hive/pull/6218#discussion_r2610896942
##########
data/conf/iceberg/llap/hive-site.xml:
##########
@@ -397,4 +397,9 @@
<name>hive.lock.sleep.between.retries</name>
<value>2</value>
</property>
+
+ <property>
+
<name>iceberg.catalog-default.write.metadata.delete-after-commit.enabled</name>
Review Comment:
How about:
1. iceberg.table-default.* - NEW
- Scope: Global (all catalogs)
- Purpose: Default table properties applied when creating new tables
- Notes / Precedence: Lowest precedence; can be overridden by
catalog-specific table-defaults or explicit table properties
2. iceberg.catalog-default.* - EXISTING
- Scope: Default catalog
- Purpose: Fallback for catalog-level settings when no named catalog
properties are configured
- Notes / Precedence: Applied when no named catalog configurations exist
3. iceberg.catalog.<name>.* - EXISTING
- Scope: Named catalog
- Purpose: Configures specific catalog settings (warehouse, type, lock
implementation, etc.)
- Notes / Precedence: Overrides catalog-default for this catalog
4. iceberg.catalog.<name>.table-default.* - NEW
- Scope: Table defaults for a named catalog
- Purpose: Default table properties for tables created in this catalog
- Notes / Precedence: Overrides global table-default.* (edited)
--
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]
