None of these calls should throw an exception (since [`THREADS > 0`](http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/Executors.html#newFixedThreadPool\(int\)), so move this out of the `try` block? And then we don't need the `if (executor != null)...` check in the `finally` below..?