Legacy Lock Thread Names Can Cause Confusion

[jeremyk-91]

Suppose I have an unowned Executor that I pass to a LockServiceImpl. This will spawn two tasks, the token and grant reaper, that will rename LockServiceImpl threads. However, when the LockServiceImpl is shut down those threads are not restored to their original names. If someone decides to re-use them, they will have that name.