AHeise commented on pull request #17640: URL: https://github.com/apache/flink/pull/17640#issuecomment-961928372
> @AHeise @MartijnVisser 2 general questions: > > 1. the current PR is a port of [this PR](https://github.com/apache/flink/pull/17523/files) for 1.13 that was asked by @AHeise, but some of the formats were removed in 1.14, we should not include them to master/1.14, right ? Yes only the ones that are currently supported in master. > > 2. the review comments of the current PR also apply to [this already merged PR](https://github.com/apache/flink/pull/17523/files) as the content is identical. I should apply them to the other PR then right ? Ideally, we would backport it again. But for me, it's more important to improve on the master. It depends a bit on how severe the improvements are. -- 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]
