[GitHub] [iceberg] snazy commented on pull request #7469: Core, Spark: Add configuration to control case sensitivity of CachingCatalog

["snazy (via GitHub)" <gi...@apache.org>]

snazy commented on PR #7469:
URL: https://github.com/apache/iceberg/pull/7469#issuecomment-1535323466

   This change breaks the Nessie catalog, which is case sensitive.
   
   `SparkUtil.caseSensitive` returns `false`, if the `spark.sql.caseSensitive` is not set, which is IMO not the right behavior for case-sensitive catalogs. Also not sure whether it's correct to imply that a _catalog_ is case sensitive (or not) from `spark.sql.caseSensitive`, because AFAIU that property is used for the table schema (case (in)sensitive column names).
   


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org