Details
Minor Description
Virtualhost tasks should reset their thread name upon completion should reset their thread name upon completion.
We use a ScheduledThreadPoolExecutor for running Virtualhost level tasks such as TransactionTimeout, VirtualHostHouseKeepingTask, the UpdateTask, etc. These rename the thread at the start of their execution, but do not at the end (as explicitly noted in the code, remove the comment) because it will be reset at the next tasks execution. This can confusingly result in multiple threads with the same name if the pool allocates a task to different threads on subsequent executions, so the tasks should restore the original name when they complete to avoid confusion.
This will result in the threads always having their generic thread pool name except when executing a given task. It might be useful if we could set the base name for these threads in some way to make them easier to identify, but this may not actually be possible while using a ScheduledThreadPoolExecutor.