> +import com.google.common.base.Objects;
> +
> +import java.beans.ConstructorProperties;
> +
> +/**
> + * Information on external gateway for the router
> + *
> + * @author Nick Livens
> + */
> +public class ExternalGatewayInfo {
> +
> +   private final String networkId;
> +
> +   @ConstructorProperties({"network_id"})
> +   protected ExternalGatewayInfo(String networkId) {
> +      this.networkId = networkId;

That seems sensible enough, but it differs from what we do elsewhere in 
OpenStack, I think, where null checks are indeed added even though (I guess) 
the values "should never" be null.

@everett-toews: thoughts on this?

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

Reply via email to