jackylee-ch opened a new pull request, #895: URL: https://github.com/apache/paimon-rust/pull/895
`lumina.distance.metric` accepts the enum spellings (`L2`) as well as the native ones, but the raw string was copied into the native option map that `LuminaIndexBuildBuilder` commits as index metadata. `LuminaIndexMeta::metric()` decodes with exact-match `from_lumina_name`, so every query against such an index failed with `Unknown lumina metric name: L2`. Fixed by persisting `metric.lumina_name()`, as Java does (apache/paimon#8676). For reviewers: the `strip_lumina_options` call is not redundant. `LuminaVectorReader::ensure_loaded` overlays the stripped table options on top of the metadata before creating the searcher, so the configured spelling would otherwise be what crosses the FFI, not the one the index was built with. Readers stay exact-match and an unrecognized value passes through untouched, both pinned by tests. An index already committed with an enum spelling needs a rebuild. -- 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]
