keith-turner opened a new pull request, #4840:
URL: https://github.com/apache/accumulo/pull/4840
Changes in this commit :
* Scan sessions with an associated running thread will continue to show up
in listscans. Previously they would stop showing up after an attempt was made
to interrupt the thread.
* When a scan session is removed and it has an associated runing thread
then it will be interrupted. This is in addition to setting the interrupt
atomic boolean. Previously only the atomic boolean was set.
* A new metric will count removed scan session that still have a running
thread. Scan session are removed when the client is no longer interested in
the result. The new metric name is `accumulo.scan.zombie.threads`.
* A new test that checks the new metric is working, listscans shows zombie
scans, and that stuck threads are interrupted.
* Scan sessions can have an associated scans task. Scan task can have an
associated thread. Added checks to ensure the scan task for scan session is
not cleared if it has a thread. This helps ensure the zombie thread detection
works. Also it is a good general improvement as clearing the task while it has
an active thread could result in silent data loss. Does not seem like the
current code would ever violate this check.
--
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]
