keith-turner commented on a change in pull request #2346:
URL: https://github.com/apache/accumulo/pull/2346#discussion_r746930818



##########
File path: 
core/src/main/java/org/apache/accumulo/core/clientImpl/ScannerIterator.java
##########
@@ -65,9 +67,9 @@
 
   private ScannerImpl.Reporter reporter;
 
-  private static ThreadPoolExecutor readaheadPool = 
ThreadPools.createThreadPool(0,

Review comment:
       > So maybe nothing needs to be done here. The core pool size is already 
being set to zero.
   
   Could possibly change the max from max int to 1.  W/ the synchronous queue I 
think it would throw an exception if more than one task was added to the pool, 
which if I Am reading the scanner code correctly is not expected.




-- 
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