Re: Review Request 22940: Return field host_components fully when hosts filtered (Hosts table's filtering by Host Components won't work without this)

[Dmytro Sen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22940/
-----------------------------------------------------------

(Updated June 25, 2014, 1:43 p.m.)


Review request for Ambari, Sid Wagle and Tom Beerbower.


Bugs: AMBARI-6262
    https://issues.apache.org/jira/browse/AMBARI-6262


Repository: ambari


Description
-------

Follow request:
{noformat}
/api/v1/clusters/<cluster-name>/hosts?host_components/HostRoles/component_name.in(<component-names>)&fields=host_components
{noformat}
should return host_components field with all host-components on the host, not 
just ones that in filter.


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java 
63426cb 
  
ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java
 ad453d9 

Diff: https://reviews.apache.org/r/22940/diff/


Testing
-------

[INFO] Reactor Summary:
[INFO] 
[INFO] Ambari Main ....................................... SUCCESS [1.632s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.303s]
[INFO] Ambari Web ........................................ SUCCESS [22.921s]
[INFO] Ambari Views ...................................... SUCCESS [1.546s]
[INFO] Ambari Server ..................................... SUCCESS [14:21.956s]
[INFO] Ambari Agent ...................................... SUCCESS [6.150s]
[INFO] Ambari Client ..................................... SUCCESS [0.035s]
[INFO] Ambari Python Client .............................. SUCCESS [0.291s]
[INFO] Ambari Groovy Client .............................. SUCCESS [8.195s]
[INFO] Ambari Shell ...................................... SUCCESS [4.434s]


Thanks,

Dmytro Sen