ajothomas commented on code in PR #1679:
URL: https://github.com/apache/samza/pull/1679#discussion_r1283830982
##########
samza-core/src/main/java/org/apache/samza/task/DefaultTaskExecutorFactory.java:
##########
@@ -50,9 +50,8 @@ public ExecutorService getTaskExecutor(Config config) {
* {@inheritDoc}
*
* The choice of thread pool is determined based on the following logic
- * If job.operator.thread.pool.enabled,
- * a. Use {@link #getTaskExecutor(Config)} if
job.container.thread.pool.size > 1
- * b. Use default single threaded pool otherwise
+ * 1. Use {@link #getTaskExecutor(Config)} if
job.container.thread.pool.size > 1
Review Comment:
Nit: This change seems unrelated and doesn't affect build. Do we need this
with this PR?
--
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]
