milleruntime commented on pull request #2279:
URL: https://github.com/apache/accumulo/pull/2279#issuecomment-924021561


   I was able to simulate a long running scan using SlowIterator with these two 
shell commands:
   <pre>
   config -t test_ingest -s 
table.iterator.scan.slow=1,org.apache.accumulo.test.functional.SlowIterator
   config -t test_ingest -s table.iterator.scan.slow.opt.sleepTime=1000
   </pre>
   
   I was wrong about this chunk of code. It isn't thread safe but it does only 
remove inactive tservers from the `allScans` map. It sets the `fetched` value 
every time it gets an active scan for that tserver. So when there are no more 
active scans for that tserver, it gets dropped from `allScans`.


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