fix: cargo fmt does not handle mods defined in macros

[roeap]

What changes are proposed in this pull request?

Apparently cargo fmt does not work on modules defined in macros, which we were doing in some places. I don't think this fixes all the misses we have been seeing with cargo fmt, but should fix some of them.

How was this change tested?

just formatting existing code.

[codecov]

Codecov Report

Attention: Patch coverage is 68.35443% with 25 lines in your changes missing coverage. Please review.

Project coverage is 84.08%. Comparing base (e2378a5) to head (24d3a06).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
kernel/src/engine/arrow_data.rs	51.21%	19 Missing and 1 partial ⚠️
kernel/src/engine/ensure_data_types.rs	77.27%	3 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #676      +/-   ##
==========================================
- Coverage   84.11%   84.08%   -0.03%     
==========================================
  Files          77       77              
  Lines       17779    17761      -18     
  Branches    17779    17761      -18     
==========================================
- Hits        14955    14935      -20     
- Misses       2110     2114       +4     
+ Partials      714      712       -2     

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

[scovich]

It all looks good, but I don't know the answer to the question about the arrow-expression vs. engine feature configs.