[GitHub] [flink] zoltar9264 commented on pull request #21822: [FLINK-30863][state] Register local recovery files of changelog before notifyCheckpointComplete()

2023-04-27 Thread via GitHub
zoltar9264 commented on PR #21822: URL: https://github.com/apache/flink/pull/21822#issuecomment-1524782057 LGTM, @Myasuka Do you have any other comments? -- 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

[GitHub] [flink] zoltar9264 commented on pull request #21822: [FLINK-30863][state] Register local recovery files of changelog before notifyCheckpointComplete()

2023-04-16 Thread via GitHub
zoltar9264 commented on PR #21822: URL: https://github.com/apache/flink/pull/21822#issuecomment-1510596889 Hi @Myasuka , may I request you help us review this pr ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [flink] zoltar9264 commented on pull request #21822: [FLINK-30863][state] Register local recovery files of changelog before notifyCheckpointComplete()

2023-04-05 Thread via GitHub
zoltar9264 commented on PR #21822: URL: https://github.com/apache/flink/pull/21822#issuecomment-1498381893 Hi @curcur and @rkhachatryan , can you help us review this pr ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [flink] zoltar9264 commented on pull request #21822: [FLINK-30863][state] Register local recovery files of changelog before notifyCheckpointComplete()

2023-03-26 Thread via GitHub
zoltar9264 commented on PR #21822: URL: https://github.com/apache/flink/pull/21822#issuecomment-1484108168 LGTM @fredia , thanks for your work. -- 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