dajac commented on PR #14728: URL: https://github.com/apache/kafka/pull/14728#issuecomment-1810000659
I am not sure to fully understand your point. You cannot change the granularity without changing how events are created in the first place. Moreover, CoordinatorEventProcessor is not right to any accumulators. It is an interface allowing you to implement a strategy to execute the events. You can use a thread pool, run them in the executor thread, or however you want. So this seems to be the right way to change how events are executed while keeping it pluggable, isn’t it? It would be great if we could respect the abstractions in place. My point is that I would be happy to merge another CoordinatorEventProcessor implementation because we can then choose the one that we want to use, based on benchmarks. If you are not interested in doing this, I would suggest to close this PR because it won’t go anywhere. -- 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]
