[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-31 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-1003372339 Sure, have a great vacation and happy new year everyone ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-30 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-1003253983 @fapaul Hi Paul, I've already rebased the branch~ -- 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] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-22 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-123030 > > @PatrickRen Updated ~ > > @imaffe > > The text used in the old JUnit4 should be more like an error msg than description before the test. It should be better to

[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-21 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-998833972 @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

[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-19 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-997651179 @PatrickRen Updated ~ -- 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

[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-12 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-992065075 @JingGe Is there anything else we might missed ? I feel this PR is about ready. WDYT ? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [flink] imaffe commented on pull request #18014: [FLINK-24857][test][Kafka] Upgrade SourceReaderTestBase t…

2021-12-08 Thread GitBox
imaffe commented on pull request #18014: URL: https://github.com/apache/flink/pull/18014#issuecomment-989443748 @JingGe Updated ~ -- 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