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:49 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
-----

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

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


Testing (updated)
-------

[INFO] Reactor Summary:
[INFO] 
[INFO] Ambari Main ....................................... SUCCESS [1.822s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.314s]
[INFO] Ambari Web ........................................ SUCCESS [23.185s]
[INFO] Ambari Views ...................................... SUCCESS [1.573s]
[INFO] Ambari Server ..................................... SUCCESS [14:21.214s]
[INFO] Ambari Agent ...................................... SUCCESS [3.588s]
[INFO] Ambari Client ..................................... SUCCESS [0.032s]
[INFO] Ambari Python Client .............................. SUCCESS [0.267s]
[INFO] Ambari Groovy Client .............................. SUCCESS [8.225s]
[INFO] Ambari Shell ...................................... SUCCESS [4.582s]


Thanks,

Dmytro Sen