shyjsarah opened a new pull request, #623: URL: https://github.com/apache/paimon-rust/pull/623
### Purpose Linked issue: close #622 `SQLContext` applies session-scoped Paimon dynamic options to ordinary table scans, but the automatically registered `vector_search` function loaded its table directly from the raw catalog. This caused options such as `blob-as-descriptor` to be ignored when vector-search results were materialized. ### Brief change log - pass the shared `DynamicOptions` instance when `SQLContext` registers `vector_search` - merge a snapshot of the current options into the loaded table before creating the vector-search provider - preserve the existing standalone `register_vector_search` and `VectorSearchFunction::new` APIs by using empty options there - add a regression test proving that an option set after catalog registration reaches the vector-search table ### Tests - `cargo fmt --all -- --check` - `cargo check -p paimon-datafusion --all-features` - `cargo clippy -p paimon-datafusion --all-features --lib --tests --no-deps -- -D warnings` - `cargo test -p paimon-datafusion test_vector_search_applies_session_dynamic_options --lib` - `cargo test -p paimon-datafusion --test read_tables vector_search_tests::test_vector_search_java_vindex_table` The full `paimon-datafusion --lib` run completed 311 tests successfully; 7 existing warehouse-fixture tests failed because the local `PAIMON_TEST_WAREHOUSE` did not contain `partitioned_log_table` / `multi_partitioned_log_table`. ### API and Format No public API or storage-format changes. ### Documentation No documentation changes are required; this makes `vector_search` consistent with the documented session dynamic-option behavior. -- 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]
