fix: adhere write stats configuration

[ion-elgreco]

Description

@roeap according to the protocol we should only write struct stats when enabled. Currently we were ignoring this config and always writing struct stats. The table config is now adhered when either of two are disabled.

• closes Adhere to delta.checkpoint.writeStatsAsStruct #3207

[codecov]

Codecov Report

Attention: Patch coverage is 90.78947% with 28 lines in your changes missing coverage. Please review.

Project coverage is 72.23%. Comparing base (1befab9) to head (a44fc9c).

Files with missing lines	Patch %	Lines
crates/core/src/protocol/checkpoints.rs	48.38%	15 Missing and 1 partial ⚠️
crates/core/src/kernel/arrow/mod.rs	95.60%	9 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3209      +/-   ##
==========================================
+ Coverage   72.14%   72.23%   +0.08%     
==========================================
  Files         138      138              
  Lines       45394    45643     +249     
  Branches    45394    45643     +249     
==========================================
+ Hits        32751    32968     +217     
- Misses      10562    10598      +36     
+ Partials     2081     2077       -4     

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

[roeap]

@ion-elgreco one question :).

We can certainly merge this fix the way it is implemented now, but ...

delta-kernel-rs contains some nice facilities to transform schemas in a more structured manner. In some cases implementations can be very simple.

https://github.com/delta-io/delta-kernel-rs/blob/eedfd473f334ac282d23847110f282118b2203eb/kernel/src/scan/data_skipping.rs#L79-L90

While a little more work, this could also brings us forward in adopting kernel etc and simplifying our codebase :).

Using this can of course also be deferred to a later PR.

[ion-elgreco]

@roeap I can look into that later for sure in a separate PR, but let's merge this to solve some current inconsistenies

[ion-elgreco]

@roeap can we merge this?