nodece commented on PR #24714: URL: https://github.com/apache/pulsar/pull/24714#issuecomment-3273505754
> > but you still need to handle the fact that `canLookup`, `canConsume`, and `canProduce` are synchronized methods, since they are public. > > Confused about handling synchronized methods `canLookup`, `canConsume`, and `canProduce`. Only one place uses `canLookup`, `canConsume`, and `canProduce` are not used in Pulsar(except in unit tests). How do I handle these three synchronized methods since I don't know where and how they will be called. You can add the `@deprecated` to these methods, and improve the `get(int, TimeUnit)` in the synchronized method. -- 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]
