kbendick commented on issue #4624: URL: https://github.com/apache/iceberg/issues/4624#issuecomment-1136244147
Hi @yittg! Are you still planning on trying to get this into 1.14? I personally think it's dangerous to do, and that we should avoid it. It would only work for the newest versions(s) of Flink 1.14, and we don't have control over which version is in use. We also don't run tests across different patch releases of any framework. If you're not, please close this issue. If you would like to patch this into 1.14 conditionally, we'll have to see what other people think as this change could only be applied to Flink versions 1.14.5+ and I don't believe there is any existing precedence for code being changed at the patch level (just the minor version level). Let us know what you think! -- 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]
