You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/01/04 04:27:37 UTC

[GitHub] [flink] RocMarshal commented on pull request #15140: [FLINK-20628][connectors/rabbitmq2] RabbitMQ connector using new connector API

RocMarshal commented on pull request #15140:
URL: https://github.com/apache/flink/pull/15140#issuecomment-1004521167


   Thank you @SteNicholas @fapaul .
   If it is determined that it will be be merged, IMO, I would do the table source and sink work based on this PR, which will enable the two PR to undertake. Instead of making some changes again for intermediate changes when refusing this PR.
   So, we only need to focus on whether the current PR can be merged to decide whether we can continue the next work based on this PR. Please let me know what's your opinion.


-- 
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-unsubscribe@flink.apache.org

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