shyjsarah commented on code in PR #482:
URL: https://github.com/apache/paimon-rust/pull/482#discussion_r3556882683
##########
crates/integrations/datafusion/src/sql_context.rs:
##########
@@ -649,6 +669,37 @@ impl SQLContext {
}
}
+ async fn load_table_for_read(
Review Comment:
Fixed in `fc2f5f9`. I extracted the branch-aware loading logic into a shared
helper and reused it in `full_text_search`, `vector_search`, `hybrid_search`,
and `blob_view`.
I also updated the full-text and vector search builders to read snapshots
through `Table::snapshot_manager()`, while manifest paths still resolve from
the table root.
Regression tests were added for all four read entry points. The full-text
and vector tests remove the main snapshot directory to verify that branch reads
do not fall back to the main branch. All CI checks are passing.
--
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]
