rdblue commented on pull request #4193: URL: https://github.com/apache/iceberg/pull/4193#issuecomment-1049192327
If I understand correctly, the idea here is that because this is binary compatible using Flink's Java API, we don't need to care about what version of Scala was used in the Flink artifact. I don't think that I agree because if you rely on the Iceberg Flink Jar, it could transitively pull in `flink-scala_2.12`. I think that the runtime Jar could be published without the Scala suffix since it has no transitive dependencies. Or we could remove the transitive dependency and remove the Scala version suffix from all the Jars. But I don't think that Iceberg should public a Jar that has no Scala 2.12 suffix, but then pulls in a Scala 2.12 library. -- 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]
