[GitHub] [flink] fredia commented on pull request #20313: [FLINK-28597][state] Discard initial checkpoints without a delay in common cases

2022-08-01 Thread GitBox
fredia commented on PR #20313: URL: https://github.com/apache/flink/pull/20313#issuecomment-1201962209 Thanks a lot for updating the PR @rkhachatryan, the changes LGTM, could you please squash the commits? -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [flink] fredia commented on pull request #20313: [FLINK-28597][state] Discard initial checkpoints without a delay in common cases

2022-07-26 Thread GitBox
fredia commented on PR #20313: URL: https://github.com/apache/flink/pull/20313#issuecomment-1195202098 Except for [`testUnregisterNonInitialCheckpoint`](https://github.com/apache/flink/pull/20313#discussion_r927001242), this PR looks good to me. @curcur would you like to take a look