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

   ### Purpose
   
   Fix the broken build on `main`. #479 (incremental global index build) was 
authored/reviewed on a base *before* #478 (Java-compatible bitmap global index) 
landed. #478 renamed the btree index-type constant from `BTREE_INDEX_TYPE` to 
`BTREE_GLOBAL_INDEX_TYPE` and moved it into `table::global_index_types`, 
removing the old definition from `btree_global_index_build_builder.rs`.
   
   The squash-merge of #479 onto the now-#478 `main` merged textually but left 
four `BTREE_INDEX_TYPE` references whose definition no longer exists, breaking 
the build:
   
   ```
   error[E0425]: cannot find value `BTREE_INDEX_TYPE` in this scope
     --> crates/paimon/src/table/btree_global_index_build_builder.rs:116, 139, 
1362, 1752
   ```
   
   Neither #478 nor #479 CI caught it because each passed against its own base; 
the incompatibility only exists in the merged tree.
   
   ### Brief change log
   
   - Point the four `BTREE_INDEX_TYPE` references in 
`btree_global_index_build_builder.rs` at `BTREE_GLOBAL_INDEX_TYPE` (already 
imported at the top of the file from `super::global_index_types`, and already 
used elsewhere in the same file). No behavioral change — same `"btree"` string 
value.
   
   ### Tests
   
   - `cargo build -p paimon` succeeds (was failing on main).
   - `cargo test -p paimon --lib` 1282 passed / 1 ignored.
   - `cargo fmt --all --check` and `cargo clippy -p paimon --all-targets -- -D 
warnings` clean.
   
   ### API and Format
   
   No API or format change. Pure merge-residual fix.
   
   ### 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