feat(fulltext): read global indexes via paimon-ftindex-core#571
Draft
shyjsarah wants to merge 1 commit into
Draft
feat(fulltext): read global indexes via paimon-ftindex-core#571shyjsarah wants to merge 1 commit into
shyjsarah wants to merge 1 commit into
Conversation
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.
Purpose
Migrate the existing append/data-evolution global full-text search path from the legacy in-repository Tantivy implementation to the shared
paimon-ftindex-coreengine.This lets Rust and DataFusion read the current
full-textindex archives produced by Java Paimon and PyPaimon while preserving the existingfull_text_search(table_name, column_name, query_text, limit)API.This PR intentionally focuses on the non-primary-key/global-index path. The primary-key full-text and hybrid search work is covered separately by #568.
Brief change log
paimon-ftindex-core.full-textentries from snapshot index manifests through a bounded-concurrencySeekReadadapter.fast,full, anddetailsearch modes with Java behavior, including raw fallback for uncovered row ranges.Testing
cargo check -p paimon --features fulltextcargo test -p paimon --features fulltext full_text -- --nocapturecargo check -p paimon-datafusion --features fulltextcargo test -p paimon-datafusion --features fulltext fulltext_tests -- --nocapturecargo clippy -p paimon --lib --tests --features fulltext -- -D warningscargo clippy -p paimon-datafusion --lib --tests --features fulltext -- -D warningsAPI and format
fulltextfeature now reads the current Paimonfull-textarchive format. Compatibility with the legacy paimon-rusttantivy-fulltextarchive is intentionally removed.Draft status and coordination
paimon-ftindex-coreis temporarily pinned to the publicv0.1.0-rc5Git tag. The dependency will be changed to a crates.io version before this PR is marked ready, because a git-only dependency prevents publishingpaimon.This work overlaps with #563 in dependency and reader wiring. It is being opened early so we can agree on the integration order and reuse or rebase on the reader foundation rather than duplicate it. It complements, but does not implement, the primary-key full-text and hybrid search scope in #568.