> +                     logger.warn(e, "error finding drive %s: %s", input, 
> e.getMessage());
> +                  }
> +                  return Futures.immediateFuture(null);
> +               }
> +
> +               @Override
> +               public String toString() {
> +                  return "seedDriveCache()";
> +               }
> +            }, userExecutor, null, logger, 
> "drives")).filter(PREINSTALLED_DISK);
> +   }
> +
> +   @SuppressWarnings("unchecked")
> +   @Override
> +   public Iterable<ServerInfo> listNodes() {
> +      return (Iterable<ServerInfo>) client.listServersInfo();

Unnecessary cast (and warning suppression)? The method already returns a List, 
which is Iterable.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/12/files#r5278539

Reply via email to