Conditionally wrap Thread.sleep(..) call in collectResults(..).

Keep Thread.sleep(..) to avoid the busy (hot) loop when the Future executions are not yet complete.

Add test coverage for ClusterCommandExecutor collectResults(..) method.

Cleanup compiler warnings in ClusterCommandExecutorUnitTests.

Closes #2518

Author: jxblum