tweise commented on pull request #1956: URL: https://github.com/apache/iceberg/pull/1956#issuecomment-759641805
@stevenzwu if Flink 1.12 is required to for the source implementation, then it should be fine to make that the minimum Flink version for the next release. Flink 1.12.1 voting is underway and the X.Y.1 version is typically what gets adopted downstream, X.Y.0 would be a bit premature for this purpose IMO. Is the long term plan to contribute the connector to Flink? If so, the versioning problem goes away and maybe does not need to be addressed within Iceberg (depending how long it will take to contribute to Flink). The Beam Flink version support approach has worked generally well. Even if only one of the versions is available in the IDE, that is normally sufficient for development. ---------------------------------------------------------------- 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]
