----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30188/#review69261 -----------------------------------------------------------
Ship it! ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java <https://reviews.apache.org/r/30188/#comment113904> when would we have a user predicate for non-collection resource? - John Speidel On Jan. 22, 2015, 8:49 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30188/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 8:49 p.m.) > > > Review request for Ambari and John Speidel. > > > Bugs: AMBARI-8342 > https://issues.apache.org/jira/browse/AMBARI-8342 > > > Repository: ambari > > > Description > ------- > > GET http://AMBARI_HOST:8080/api/v1/clusters/c1/config_groups > > { > "status" : 404, > "message" : "The requested resource doesn't exist: ConfigGroup not found, > ConfigGroup/cluster_name=c1" > } > > The cluster c1 exists and has no configuration groups. The above response is > incorrect. As per the api specification, it should return an empty > collection. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java > 9f1be06 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java > b075d25 > > Diff: https://reviews.apache.org/r/30188/diff/ > > > Testing > ------- > > In progress ... tests to be added > > > Thanks, > > Tom Beerbower > >
