SEZ9 commented on issue #12707: URL: https://github.com/apache/seatunnel/issues/12707#issuecomment-6116592198
@karthik-0306 Sounds good — that plan matches what we discussed, and the three points you listed are exactly what I'm looking for: - A standalone module exposing a small function that returns the resolved Java executable path and parsed major version, so the launcher can reuse it. - pytest coverage for both the legacy `1.8.0_x` format and the newer `11` / `17+` `java -version` output, with the command output mocked. - A clear error when Java is missing or below 11 that says what was found (or that nothing was found) and points to setting JAVA_HOME or installing JDK 11+. Nothing else needed from my side right now. Go ahead and open the PR, link it back to this issue, and I'll review it there. If the overall design shifts while you're working on it, just note the change in the PR description so we can adjust together. <!-- streview-comment:1686 --> -- 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]
