@wForget fyi

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 58.56%. Comparing base (f09f8af) to head (c592764).
Report is 213 commits behind head on main.

@@             Coverage Diff              @@
##               main    #1782      +/-   ##
============================================
+ Coverage     56.12%   58.56%   +2.43%     
- Complexity      976     1140     +164     
============================================
  Files           119      130      +11     
  Lines         11743    12718     +975     
  Branches       2251     2371     +120     
============================================
+ Hits           6591     7448     +857     
- Misses         4012     4079      +67     
- Partials       1140     1191      +51     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

This PR reads it from the SQLConf, serializes it to the native side, and stashes it in SparkParquetOptions. I don't see what happens with it after that. Presumably we could then use this info in the SchemaAdapter but I don't see that logic anywhere. I think it's just a dead-end config like is_adapting_schema right now.

This PR reads it from the SQLConf, serializes it to the native side, and stashes it in SparkParquetOptions. I don't see what happens with it after that. Presumably we could then use this info in the SchemaAdapter but I don't see that logic anywhere. I think it's just a dead-end config like is_adapting_schema right now.

We do have code that uses this config:

                    if self.parquet_options.case_sensitive {
                        b.name() == field.name()
                    } else {
                        b.name().to_lowercase() == field.name().to_lowercase()
                    }

The newly added test was previously failing for native_datafusion (but not for native_iceberg_compat for some reason).