> + */
> +package org.jclouds.openstack.nova.v2_0.domain.zonescoped;
> +
> +import com.google.common.base.Objects;
> +import java.beans.ConstructorProperties;
> +
> +/**
> + * @author Inbar Stolberg
> + */
> +public class AvailabilityZone {
> +
> +   private final String name;
> +   private final ZoneState zoneState;
> +
> +   @ConstructorProperties({
> +         "zoneName" , "zoneState"

If I am not wrong, the current values for the `@ConstructorProperties` are the 
right ones. At least the ones jclouds expects. Those values are used to 
deserialize the response. The 
[GsonModule](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/json/config/GsonModule.java#L123-L124)
 configures a [specific 
strategy](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/json/internal/NamingStrategies.java#L200-L250)
 to deserialize responses when the constructor has arguments.

So the values in the annotation are the right ones to get the response properly 
deserialized, but perhaps the name of the field and the constructor parameters 
could be changed to keep a more purist naming convention.

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

Reply via email to