> + 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;
Should we be making immutable copies of the input sets here?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/22/files#r18015255
