> +   private final Set<URI> healthChecks;
> +   private final Set<URI> instances;
> +   private final Optional<String> sessionAffinity;
> +   private final float failoverRatio;
> +   private final Optional<String> backupPool;
> +
> +   @ConstructorProperties({
> +           "id", "creationTimestamp", "selfLink", "name", "description", 
> "region", "healthChecks", "instances",
> +           "sessionAffinity", "failoverRatio", "backupPool"
> +   })
> +   private TargetPool(String id, Date creationTimestamp, URI selfLink, 
> String name, String description,
> +                      URI region, Set<URI> healthChecks, Set<URI> instances, 
> String sessionAffinity, float failoverRatio, String backupPool) {
> +      super(Kind.TARGET_POOL, id, creationTimestamp, selfLink, name, 
> description);
> +      this.region = checkNotNull(region, "region of %s", name);
> +      this.healthChecks = healthChecks == null ? ImmutableSet.<URI>of() : 
> healthChecks;
> +      this.instances = instances == null ? ImmutableSet.<URI>of() : 
> instances;

Can these two ever be `null`? The builder will always call them with a non-null 
value...

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

Reply via email to