> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + ResourceGroup get(@PathParam("name") String name);
> +
> + @Named("resourcegroup:update")
> + @PATCH
> + @Produces(MediaType.APPLICATION_JSON)
> + @Path("resourcegroups/{name}")
> + @Fallback(NullOnNotFoundOr404.class)
> + @Nullable
> + @MapBinder(BindToJsonPayload.class)
> + ResourceGroup update(@PathParam("name") String name,
> @PayloadParam("tags")Map<String, String> tags);
> +
> + @Named("resourcegroup:delete")
> + @DELETE
> + @ResponseParser(StatusCodeParser.class)
What is the purpose of returning the response status? (And in that case, why
not returning an int?). If the purpose is to let users know if the operation
succeeded, I'd suggest changing this method to return a `boolean` with the
`FalseOnNotFoundOr404` fallback. This way the method will return `true` if the
delete operation succeeds, or throw an exception or return `false` if it fails.
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/250/files/f7cc1b437f11378e105bba552e3cfee9d7491872#r57578717
