TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1576310784
Thanks @xintongsong and @reswqa for the careful review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1575978610
I rebased on the master branch to trigger tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1573354370
@flinkbot run azure
--
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
TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1573304176
@reswqa Thanks for helping review, I have addressed the comments. Could you
help take a look again?
Due to the significant code changes in the test code, I didn't use a fix-up
TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1571493540
@xintongsong Thanks for the careful review. I have addressed the comments
with two fix-up commits.
--
This is an automated message from the Apache Git Service.
To respond to the
TanYuxin-tyx commented on PR #22652:
URL: https://github.com/apache/flink/pull/22652#issuecomment-1569708841
@flinkbot run azure
--
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