Kam

Merge current REST work into master
It is necessary to handle no args (including PathParam). I thought 400 was more accurate (Bad Request) vs (Not Found) in order to provide specific information about incorrect or missing args (including possible required QueryParams).


On Tuesday, August 5, 2014 2:48 PM, Sean Owen <notifications@github.com> wrote:
 


In oryx-ml-oryx-serving/src/main/java/com/cloudera/oryx/ml/serving/als/Recommend.java:
> @@ -41,6 +44,12 @@
>  public final class Recommend {
>  
>    @GET
> +  @Produces(MediaType.APPLICATION_JSON)
> +  public Response get() { 
Is this a placeholder or is it necessary to handle the case of no args? I'm OK with it generating a 404 instead of having to write this to generate a 400. 404 is more accurate really
—
Reply to this email directly or view it on GitHub.