oneby-wang commented on PR #24714:
URL: https://github.com/apache/pulsar/pull/24714#issuecomment-3266396219

   > Similar changes in #24383. I don't think that we should blindly change 
from thenComposeAsync to thenCompose since some implementations might be 
blocking the callback completion thread. This change might break some 
implementations.
   
   I see, I think at least we should specify an executor(created by netty 
DefaultThreadFactory) for thenComposeAsync 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: commits-unsubscr...@pulsar.apache.org

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

Reply via email to