[GitHub] [flink] fredia commented on pull request #21895: [FLINK-30792][state/changelog] make state changelog pre-emptively upl…

2023-02-09 Thread via GitHub


fredia commented on PR #21895:
URL: https://github.com/apache/flink/pull/21895#issuecomment-1425101525

   Thanks for the update, CI test failed because of 
https://issues.apache.org/jira/browse/FLINK-30972, maybe we can rebase this 
into PR.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [flink] fredia commented on pull request #21895: [FLINK-30792][state/changelog] make state changelog pre-emptively upl…

2023-02-08 Thread via GitHub


fredia commented on PR #21895:
URL: https://github.com/apache/flink/pull/21895#issuecomment-1423534579

   @zoltar9264 Thanks for the PR, overall it LGTM. I left some minor comments 
about the test, PTAL, thanks.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org