gaborgsomogyi commented on PR #29422:
URL: https://github.com/apache/flink/pull/29422#issuecomment-6210314014

   Thanks for removing the sentence. Please keep it as `not supported` and 
don't document the drain + changed sink uid + `--allowNonRestoredState` recipe. 
That flag silently drops any unmapped state job-wide, and if the stop ends with 
`StopWithSavepointStoppingException` it also loses the last files, so it is 
easy to lose data with it. The point of this PR is a clean path that needs none 
of that.
   
   `MiniClusterITCase` seems like unrelated, can you plz rebase to the latest 
master and force push it?
   


-- 
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]

Reply via email to