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/04/28 06:36:58 UTC

[GitHub] [flink] AHeise commented on pull request #19430: [BK-1.15][FLINK-26931][Connector/pulsar] Make the producer name and consumer name unique in Pulsar

AHeise commented on PR #19430:
URL: https://github.com/apache/flink/pull/19430#issuecomment-1111803212

   > Since this PR fixes an actual bug, I would expect to add a test case. Can you add one so that the issue does not arise again?
   
   I think this is hard to test on integration level (you probably need to check on Pulsar server if a specific producer is registered) and the change is rather implicitly tested with other tests.
   However, if there is an easy way to test, I'm also in favor of adding it.


-- 
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