SEPURI-SAI-KRISHNA commented on PR #13090: URL: https://github.com/apache/seatunnel/pull/13090#issuecomment-6568610009
Thanks @DanielLeens for the review. On Issue 1, I would keep the explicit enum: a major Elasticsearch release can change the bulk or search APIs, as 9.0 did with stricter bulk parsing, so a new major should be added together with an IT that runs against it rather than accepted silently. The `ELASTICSEARCH-02` error already names the version, so an unsupported cluster is easy to diagnose. -- 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]
