wypoon commented on code in PR #7469:
URL: https://github.com/apache/iceberg/pull/7469#discussion_r1184183581
##########
core/src/main/java/org/apache/iceberg/CatalogProperties.java:
##########
@@ -41,6 +41,11 @@ private CatalogProperties() {}
public static final boolean CACHE_ENABLED_DEFAULT = true;
+ /** Controls whether the caching catalog will cache table entries using
case-sensitive keys. */
+ public static final String CACHE_CASE_SENSITIVE = "cache.case-sensitive";
+
+ public static final boolean CACHE_CASE_SENSITIVE_DEFAULT = false;
Review Comment:
In case there are other use cases I do not know about, I am leaving
configurability in. Also, other catalogs such as `FlinkCatalog` that use the
`CachingCatalog` may choose to make use of the `cache.case-sensitive` conf.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
