xanderbailey opened a new pull request, #3093:
URL: https://github.com/apache/iceberg-rust/pull/3093

   ## Which issue does this PR close?
   
   - Working towards #2152.
   
   ## What changes are included in this PR?
   
   Extract the existing table scan construction logic into an internal 
`ScanConfig` and `build_table_scan` function so it can be reused by other scan 
builders.
   
   This is a behavior-preserving refactor. `TableScanBuilder` still resolves 
the same snapshot and schema, and manifest planning and the public API are 
unchanged.
   
   This change was extracted from #2997 to let the shared scan construction be 
reviewed independently from incremental append scan semantics.
   
   ## Are these changes tested?
   
   - `cargo fmt --all -- --check`
   - `cargo test -p iceberg scan::tests --lib` (56 passed)
   - `cargo clippy -p iceberg --lib --all-features -- -D warnings`
   
   ## AI Disclosure
   
   Codex assisted with extracting the existing scan construction logic, 
validation, and drafting this PR. I reviewed the complete diff and verified the 
tests and lint checks listed above.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to