[GitHub] [flink] cmick commented on pull request #17708: [FLINK-23696][connectors/rabbitmq] Fix RMQSourceTest.testRedeliveredSessionIDsAck

2021-11-09 Thread GitBox
cmick commented on pull request #17708: URL: https://github.com/apache/flink/pull/17708#issuecomment-964622552 @fapaul Thanks for the review. Sure, backports added in #17740 and #17741 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [flink] cmick commented on pull request #17708: [FLINK-23696][connectors/rabbitmq] Fix RMQSourceTest.testRedeliveredSessionIDsAck

2021-11-08 Thread GitBox
cmick commented on pull request #17708: URL: https://github.com/apache/flink/pull/17708#issuecomment-963655134 @fapaul Thanks, You're absolutely right here. I've added resetting the DummyContext state in 01a81f7. Now it can be run multiple times - I've executed it locally a thousand times

[GitHub] [flink] cmick commented on pull request #17708: [FLINK-23696][connectors/rabbitmq] Fix RMQSourceTest.testRedeliveredSessionIDsAck

2021-11-08 Thread GitBox
cmick commented on pull request #17708: URL: https://github.com/apache/flink/pull/17708#issuecomment-963049982 @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