JunRuiLee opened a new pull request, #471:
URL: https://github.com/apache/paimon-rust/pull/471

   ### Purpose
   
   CI on `main` is red: the `check` job fails clippy with an unused `Result` in 
`btree_global_index_build_builder.rs:562`. #454 was written before #460 made 
`ReadBuilder::with_projection` fallible and merged without a rebase check 
against it. Every open PR's merge CI now fails on the same error regardless of 
its own changes.
   
   (Hotfix for CI breakage — no linked issue, per the contribution checklist 
exception.)
   
   ### Brief change log
   
   Propagate the `Result` with `?` — `extract_index_rows` already returns 
`Result`, and a projection failure here (unknown/duplicate column) should abort 
the index build rather than silently read the full row.
   
   ### Tests
   
   `cargo clippy -p paimon --all-targets --features fulltext,vortex,mosaic -- 
-D warnings`: clean (was failing).
   `cargo test -p paimon --lib btree`: 58 passed.
   
   ### API and Format
   
   None.
   
   ### Documentation
   
   None.
   


-- 
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]

Reply via email to