> +public interface KeysApi {
> +
> +     @Named("keys:create")
> +     @PUT
> +     @Path("/{key}")
> +     Key createKey(@PathParam("key") String key, 
> +              @FormParam("value") String value, 
> +              @Nullable @FormParam("ttl") Integer seconds);
> +
> +   @Named("keys:get-key")
> +   @GET
> +   @Path("/{key}")
> +   @Fallback(NullOnKeyNonFoundAnd404.class)
> +   Key getKey(@PathParam("key") String key, 
> +            @Nullable @QueryParam("recursive") Boolean recursive,
> +            @Nullable @QueryParam("sorted") Boolean sorted);

Same as before. Allowing to pass those `(foo, null, null)` is kinda ugly. 
Consider providing an overloaded version that uses primitive types for the 
options.

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

Reply via email to