SEZ9 commented on issue #12988: URL: https://github.com/apache/seatunnel/issues/12988#issuecomment-6593625346
@SEPURI-SAI-KRISHNA thanks for the summary. The scope you listed — the `ES9` value, the serializer unit test, `Elasticsearch9IT` on `elasticsearch:9.5.5` covering insert/update/delete plus scroll, PIT and SQL reads, and the EN and ZH docs — matches what this issue needs. I'll do my review on the PR thread itself. One small ask: please add a "Fixes" reference to this issue in the PR description so the two get linked. @zhangwen0527 thanks for sharing your config (`face20261021.json` with the `face_vector` float array) and the log for job `1169512690318573569` — that is exactly the detail that was missing when this issue was opened. Once the PR is merged, could you try it against your 9.5.5 cluster with the same config and report back whether the dynamically created `t_face_vector` mapping for `face_vector` works for you, or whether you need to pre-create the index with an explicit mapping as SEPURI-SAI-KRISHNA described? I'll keep this issue open until the PR lands and we have your confirmation. <!-- streview-comment:2230 --> -- 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]
