findepi commented on PR #10477: URL: https://github.com/apache/iceberg/pull/10477#issuecomment-2160536777
> I'm open to running against Java 17. My only concern is that the Spark/Flink tests take a very long time to run, and adding another item to the test-matrix will consume a lot of CI capacity. That's a valid concern. OTOH, if we don't run the tests, how do we know that the code works under newer Java version? Of course, who knows, maybe the users never use the Iceberg Spark or Iceberg Flink code with e.g. Java 17 and therefore we don't need to support that -- is this the case? > For Iceberg 2.0 there [seems to be some (I would love to see some more folks jumping in there)](https://lists.apache.org/thread/ntrk2thvsg9tdccwd4flsdz9gg743368) consensus to bump Java 11+, maybe that would be the right time to drop Java 8 and add Java 17? Out of curiosity, did you find any issues with JDK17? I thought that Java 17 is already well supported -- the build requires you to use 8, 11 or 17. I am making this changes, as I am looking what it would take to support 21 for building too (https://github.com/apache/iceberg/pull/10474). -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
