snazy commented on pull request #2588: URL: https://github.com/apache/iceberg/pull/2588#issuecomment-850230256
> @snazy looks like the build failed for java8. Presumably its the recent chagne to nessie server to not support java8? Oh yea, https://github.com/projectnessie/nessie/pull/1245 broke the test runs. It's not that the Nessie Client in Iceberg is broken, it's "just" the server that cannot run. Reverting https://github.com/projectnessie/nessie/pull/1245 doesn't make much sense, because Quarkus 2 drops support for Java 8 entirely. I see these options here: 1. Skip the Nessie tests with "Iceberg Java 8" for now (to get this one in) 2. Change the `:iceberg-nessie` test setup to actually start the Nessie Server in a separate process using Java 11, preferably using the now published Nessie-Server uber-jar or, maybe easier, using test-containers. I think, option 1 is a good workaround for now and then follow-up with option 2 within a short time. WDYT? -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
