@shrinandj This commit represents a big improvement and I apologize for my 
delayed comments.  Can you address some of these and add TODOs for the rest so 
we can commit this as soon as possible?  Specifically we must address the O(n) 
behavior and I do not understand some of the synchronization.  Further I have 
some doubts about how the size of the ```ExecutorService``` interacts with the 
```Semaphore```; should the sizes of the two be the same?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/214#issuecomment-38323072

Reply via email to