> +
> +   @Override
> +   public Iterable<? extends Node> execute(final ListeningExecutorService 
> executor, String environmentName, Iterable<String> toGet) {
> +      ListenableFuture<List<Node>> futures = allAsList(transform(toGet, new 
> Function<String, ListenableFuture<Node>>() {
> +         @Override
> +         public ListenableFuture<Node> apply(final String input) {
> +            return executor.submit(new Callable<Node>() {
> +               @Override
> +               public Node call() throws Exception {
> +                  return api.getNode(input);
> +               }
> +            });
> +         }
> +      }));
> +
> +      logger.trace(String.format("getting nodes: %s", 
> Joiner.on(',').join(toGet)));

Worth adding the name of the environment to the log?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/13/files#r6238664

Reply via email to