codelipenghui commented on pull request #13291: URL: https://github.com/apache/pulsar/pull/13291#issuecomment-994156515
@Shoothzj I noticed the fix only fixes the compatibility issue when getting backlog quota, I think another compatibility issue is when using a new version client >= 2.8 but the broker which < 2.8, since the broker-side don't have `limitSize` and the client side will not pass the `limit` to the broker, so that the broker will use `0` as the value of `value`, it also should be a compatibility issue. It's better to add a compatibility test to make sure we will not introduce breaking change for this part. Thanks for the great work. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
