merlimat commented on a change in pull request #11387:
URL: https://github.com/apache/pulsar/pull/11387#discussion_r696888143
##########
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java
##########
@@ -252,7 +253,8 @@
protected volatile State state = null;
private final OrderedScheduler scheduledExecutor;
- private final OrderedExecutor executor;
+ private final ScheduledExecutorService pinnedScheduledExecutor;
+ private final Executor pinnedExecutor;
Review comment:
> Perhaps a more optimal solution would be to have the capability for
scheduling tasks on the pinned scheduler
@lhotari The scheduled executor is less efficient compared to the normal
executor because it has to maintain the delayed tasks. For that it's preferable
not to use it directly in the critical data path, but only when we want to
defer actions or for background tasks.
--
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]
