JingsongLi commented on PR #568: URL: https://github.com/apache/paimon-rust/pull/568#issuecomment-5065532863
- `crates/paimon/src/table/hybrid_search_builder.rs:483` directly projects all user columns without reusing the retained column checks from the vector read. - `_PKEY_VECTOR_POSITION` and `__paimon_search_score` will be appended later (`crates/paimon/src/table/pk_vector_position_read.rs:224`). - If the table itself contains a column with the same name, a positional lookup will target the user column (`crates/paimon/src/table/vector_search_builder.rs:1925`), which may result in sorting errors, deletion of the user column, and exposure of internal position columns; when the `score` column has the same name, duplicate fields will be generated, and the value read may not be the fused score. - A standalone `vector read` explicitly intercepts this pattern in `crates/paimon/src/table/vector_search_builder.rs:582`; the `hybrid` implementation should reuse the same checks and add regression tests for the two reserved columns. -- 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]
