Ferenc Csaky

@Tan-JiaLiang I think it is okay to take over. We can keep the separate commits, so the original change will keep its author, even the commit msg can be modified...

Thanks for the fix! Is this only relevant for 2.0?

@flinkbot run azure

@flinkbot run azure

Added the following PRs, which all contain the requested changes by @afedulov: - https://github.com/apache/flink/pull/25969 - https://github.com/apache/flink/pull/25970 - https://github.com/apache/flink/pull/25971 Kept @SamBarker as author. I plan to merge those 3 when all...

Thanks for the fix @gaborgsomogyi! As I tried to understand this problem, I checked the history of what happened here and why. The previous change in `S3RecoverableFsDataOutputStream#sync()` was done in...

FYI, the CI failure was not because of the `Map.of` itself, but the missed `mvn spotless:apply` for proper code format.

If the currently pending ci will be successful, i'll merge it, thanks

> Folks, let's wrap this up soon, ideally after this review round. I would definitely like to include this into 1.20.2 so that's why I jumped in, my plan is...

> I missed that I was still tagged as waiting for review from my end. I'm sorry for that. I have a few nitty comments. > > Just one concern...