Hi Xintong, Weijie,
Thank you both for your answers!
Regarding the CLI deprecation process, I agree that it is tricky
business, hence I opened this discussion. The proposal makes sense
IMO, I will open a discussion thread.
With the present timeline, I think it is reasonable to hold it
from 1.20,
+1 for Xintong's point.
Hi Ferenc,
After discussing this with all the other RM's, we decided not to merge the
PR you mentioned into 1.20.
The main reason is that FLIP-464 was voted in three days ago, which is
already after feature freeze date. It doesn't make sense to us to put it in
the 1.20 rel
Hi Ferenc,
About the deprecation process, removing a @Public API requires the API to
be deprecated for at least 2 minor releases AND the removal should be in a
major version bump. That means you cannot remove a @Public API in 2.x when
x is not 0. The tricky part is, run-application as a command-li
Hi Robert, Rui, Ufuk, and Weijie,
I would like to raise the PR about merging `flink run` and
`flink run-application` functionality [1] to get considered as
part of the 1.20 release.
The reason IMO is that the `run-application` CLI command should be
removed in the same release when Per-Job mode ge
Hi Zakelly,
Thank you for informing us!
After discussion, all RMs agreed that this was an important fix that should
be merged into 1.20.
So feel free to merge it.
Best regards,
Weijie
Zakelly Lan 于2024年6月15日周六 16:29写道:
> Hi Robert, Rui, Ufuk and Weijie,
>
> Thanks for the update!
>
> FYI:
Hi Robert, Rui, Ufuk and Weijie,
Thanks for the update!
FYI: This PR[1] fixes & cleanup the left-over checkpoint directories for
file-merging on TM exit. And the second commit fixes the wrong state handle
usage. We encountered several unexpected CI fails, so we missed the feature
freeze time. It
Hi everyone,
The feature freeze of 1.20 has started now. That means that no new features
or improvements should now be merged into the master branch unless you ask
the release managers first, which has already been done for PRs, or pending
on CI to pass. Bug fixes and documentation PRs can sti