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 2022/02/09 09:03:46 UTC

[GitHub] [pulsar] lhotari commented on pull request #14163: [Management] Make ``BrokerBase#deleteDynamicConfiguration`` to pure async method.

lhotari commented on pull request #14163:
URL: https://github.com/apache/pulsar/pull/14163#issuecomment-1033519034


   > Make BrokerBase#deleteDynamicConfiguration to a pure async method to avoid some problems caused by sync and async methods calling each other.
   
   @mattisonchao please share what these problems are. Is there a reported issue?
   Since there are a lot of these sync -> async changes, it would be good to always open a discussion on the dev-mailing list to get a consensus of the problem that is being solved. 
   Could you start a dev-mailing list thread on the topic which describes the problem, the assumptions and the solution?


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