QuakeWang opened a new pull request, #17:
URL: https://github.com/apache/paimon-vector-index/pull/17
## What changed
`IVFPQIndex::merge_from` previously only checked basic layout fields before
merging inverted lists. It did not verify that both indexes shared the same
metric, residual mode, OPQ rotation, coarse centroids, and PQ codebooks.
This could silently merge indexes trained from different states during
compaction and return incorrect ANN results.
This PR makes `merge_from` return `io::Result<()>` and rejects incompatible
training states before mutating the target index. It also adds regression tests
for metric, residual mode, OPQ, coarse centroid, and PQ codebook mismatches.
## Tests
- cargo fmt --all -- --check
- cargo clippy --all-targets --workspace -- -D warnings
- cargo build --all-targets
- cargo test --workspace
- cargo deny check licenses
- python3 tools/validate_asf_yaml.py
- cargo test --manifest-path python/Cargo.toml --no-default-features
--features auto-initialize
- cargo build -p paimon-vindex-jni --release
- Java API compile/test
--
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]
