lhotari commented on PR #17870:
URL: https://github.com/apache/pulsar/pull/17870#issuecomment-1262151799

   > we have too much historical baggage and potential impact.
   
   It would be good to clarify this if you have something specific to point 
out. I can see that there are issues with blocking code. It would be useful if 
you could point out clear examples of problems. 
   
   > Another problem is that we share the IO pool with all replicators. This 
means that if we have too many replicators on the broker, they will compete for 
resources with the broker.
   
   Sharing the IO pool isn't a problem on its own. They will compete on the 
same resources even in the case of using separate thread pools. 
   
   There's a detail in Pulsar that the BookKeeper client uses the Pulsar Broker 
IO threads by default, code at 
https://github.com/apache/pulsar/blob/df5e0e1869ff7ce55489e4a7853172fa37b2b59d/pulsar-broker/src/main/java/org/apache/pulsar/broker/BookKeeperClientFactoryImpl.java#L95-L97
 
   
   What do you think about that 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to