szehon-ho commented on PR #10472: URL: https://github.com/apache/iceberg/pull/10472#issuecomment-2158956729
Related discussion at : https://github.com/apache/iceberg/pull/8300 As a jenv user, I am not in favor of this. There is no right version here (depends on what engines you are trying to integrate Iceberg with in your org), just supported versions. While setting it to 17 is convenient for you, it is not to people trying to build different versions. This now forces a user trying to build a distribution for another java version to have to explicitly set the version in Iceberg (whereas previously they can just rely on the global setting). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
