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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2023/01/04 13:10:14 UTC

[GitHub] [pulsar] BewareMyPower commented on pull request #19128: [fix][broker] Fix consumer with schema cannot subscribe when AUTO_CONSUME consumers exist

BewareMyPower commented on PR #19128:
URL: https://github.com/apache/pulsar/pull/19128#issuecomment-1370911772

   > Please change the title, this is not only a client side (java) change but it involves the broker.
   
   Yes. I changed it to `[broker]` in the title according to the [guide](https://docs.google.com/document/d/1d8Pw6ZbWk-_pCKdOmdvx9rnhPiyuxwq60_TrD68d7BA/edit#heading=h.tqqg54v1xs21) says:
   
   > choose the most affected label (scope)
   
   > We should add (as a separate work) integration tests that verify that new clients can connect to old brokers 
   
   I agree. I'm going to check where should I add the integeration tests, maybe needs a testcontainer.


-- 
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: commits-unsubscribe@pulsar.apache.org

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