Use cached Chunk struct when considering compressed paths

[akuzm]

Full catalog lookups for a Chunk are expensive, avoiding them speeds up the planning.

Disable-check: force-changelog-file

[github-actions]

@pmwkaa, @svenklemm: please review this pull request.

Powered by pull-review

[codecov]

Codecov Report

Merging #5953 (1100ebb) into main (fb617e4) will decrease coverage by 0.53%.
The diff coverage is 88.88%.

❗ Current head 1100ebb differs from pull request most recent head cbe099e. Consider uploading reports for the commit cbe099e to get more accurate results

@@            Coverage Diff             @@
##             main    #5953      +/-   ##
==========================================
- Coverage   87.09%   86.57%   -0.53%     
==========================================
  Files         243      243              
  Lines       56018    55927      -91     
  Branches    12401    12365      -36     
==========================================
- Hits        48789    48417     -372     
- Misses       4891     5166     +275     
- Partials     2338     2344       +6     

Files Changed	Coverage Δ
src/planner/planner.h	85.71% <ø> (ø)
tsl/src/fdw/data_node_chunk_assignment.c	90.62% <50.00%> (ø)
tsl/src/planner.c	90.64% <88.88%> (-0.38%)	⬇️
src/planner/expand_hypertable.c	89.03% <100.00%> (-0.13%)	⬇️
src/planner/planner.c	90.86% <100.00%> (ø)
tsl/src/fdw/relinfo.c	93.56% <100.00%> (+0.03%)	⬆️

... and 47 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more