shyjsarah commented on PR #68: URL: https://github.com/apache/paimon-vector-index/pull/68#issuecomment-5173847102
@JingsongLi @jerry-024 Addressed the review feedback in 3d3ca6b: - Rebased onto the latest `main` (`2a8d88a`) and integrated the IVF-PQ batch table reuse/budget changes from #67. - Added the explicit Recall@K trade-off to the shared docs and Rust/Java API documentation. Progressive expansion is now documented as occurring only when fewer than `top_k` valid results are returned. - Restored the primary Rust and Java examples to plain `automatic(10)`. - Moved factor `4` to a clearly marked optional workload-specific tuning example, so the PR does not recommend it as a default or require a benchmark-backed default claim. Revalidated with workspace tests, Clippy, Maven tests, Java/JNI native validation, panic-boundary and handle-safety tests, formatting, license headers, and diff checks. -- 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]
