FANNG1 commented on PR #7418: URL: https://github.com/apache/gravitino/pull/7418#issuecomment-2996206064
> Using name `modern` is weird; you'd better use a more meaningful name. Also, can you support JDK 11+ with Iceberg 1.6? As far as I understand, I think we should support JDK 8+ with Iceberg 1.6, JDK 11+ with Iceberg 1.9, right? > > We can use something like conditional compile by specifying using which iceberg version to compile, like `gradle build -PicebergVersion=xxx`, if user chooses Iceberg 1.9, then we choose the higher version of JDK. I had thought of using an extra flag to control whether to use a new Iceberg version, but there seems to be no strong requirement for using a low Iceberg version when building with high JDK versions. -- 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]
