feat(vchordrq): estimate MaxSim planner cost - #477
Draft
HuXinjing wants to merge 1 commit into
Draft
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
Author
I have read the CLA Document and I hereby sign the CLA |
Author
|
I have read the CLA Document and I hereby sign the CLA |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
MetaTuplepadding without changing tuple offsetsThis is the first focused implementation PR proposed in #476. It does not add
an exact reranker, external tensor storage, a network protocol, or GPU code.
Compatibility
The statistic uses the existing 2-byte and 6-byte padding regions in the meta
tuple. Compile-time offset assertions and unit tests protect the on-disk layout.
None.The planner query-token and fallback document-token values are configurable
until expression/page-level statistics are available.
Validation
cargo test -p vchordrqcargo test --locked --workspace --exclude vchord --no-fail-fastxtaskbuildinside a clean PostgreSQL development container
cargo fmt --all -- --checkgit diff --checkReview notes
The cost constants are intentionally conservative and hardware-independent.
The model prices the eager token search and page aggregation currently
performed by MaxSim. A future optional exact-rerank backend can add its own cost
without changing this candidate-generation model.
Discussion: #476