neykov commented on this pull request.
> this.timeSupplier = timeSupplier; + this.nodes = Suppliers.memoize(new ConcurrentWrapperSupplier(existingMachines)); The `ConcurrentHashMap` is an implementation detail of the `VagrantNodeRegistry`. The wrapping supplier+memoize is just a mechanism to achieve lazy loading, not something that's expected to be provided by external wiring (so that we don't do IO in the constructor). That's at least my take on it, don't mind making it pluggable if you still think that'd be useful. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/355
