New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WorkerExecutor must order tasks on the current Context and not on its… #1869
Conversation
919a299
to
088ab95
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, good job! Can you please remove the commented code? Thanks
@@ -26,12 +26,12 @@ | |||
*/ | |||
class WorkerPool { | |||
|
|||
private final OrderedExecutorFactory orderedFact; | |||
// private final OrderedExecutorFactory orderedFact; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this commented code?
private final ExecutorService pool; | ||
private final PoolMetrics metrics; | ||
|
||
WorkerPool(ExecutorService pool, PoolMetrics metrics) { | ||
this.orderedFact = new OrderedExecutorFactory(pool); | ||
// this.orderedFact = new OrderedExecutorFactory(pool); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this commented code?
return orderedFact.getExecutor(); | ||
} | ||
// Executor createOrderedExecutor() { | ||
// return orderedFact.getExecutor(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove this commented code?
088ab95
to
4b32d2b
Compare
…elf - fixes #1867