As far as I can see, the main change here is to make `shellTableFactory` lazy. I'd be curious to hear what @iocanel's thoughts are on this in general, but in terms of implementation is there a reason for not having a `getShellTableFactory` method in `ComputeCommandBase` that does the "if null then instantiate" bit? Is there any reason to repeat that across many classes?
Also, does the instantiation need to be thread-safe in any way? The changes in this PR could lead to multiple instantiations. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/18#issuecomment-20357545
