nbdevos25 commented on PR #14560: URL: https://github.com/apache/iceberg/pull/14560#issuecomment-3517899413
> At first glance, the change is good. However, I have two comments: > > 1. Did you check a release is still valid ? I see a change in the generated artifacts. > 2. As I said in the GitHub Issue, Gradle 9.x requires JDK17+. AFAIR, not all parts of Iceberg are ready to JDK17+ AFAIR. We have to double check and verify with the community here. Hi @jbonofre , for #2 above, I'd appreciate if you can please lead the discussion, and please add me. I am brand new to the repo, and happy to do the work and learn. Thanks! -- 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]
