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


Ship it!




Ship It!

- DIPAYAN BHOWMICK


On May 12, 2016, 1:29 p.m., Gaurav Nagar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47306/
> -----------------------------------------------------------
> 
> (Updated May 12, 2016, 1:29 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav 
> Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev.
> 
> 
> Bugs: AMBARI-16634
>     https://issues.apache.org/jira/browse/AMBARI-16634
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Extended view Cluster interface to return hosts for service component.
> Used this method for geting the hosts for hive server, webhcat. Instead of 
> using Ambari Api in attached mode.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/view/ClusterImpl.java 
> 529e09a 
>   
> ambari-server/src/main/java/org/apache/ambari/server/view/RemoteAmbariCluster.java
>  f661844 
>   
> ambari-server/src/test/java/org/apache/ambari/server/view/ClusterImplTest.java
>  daf87ec 
>   
> ambari-server/src/test/java/org/apache/ambari/server/view/RemoteAmbariClusterTest.java
>  ce8fe7d 
>   ambari-views/src/main/java/org/apache/ambari/view/cluster/Cluster.java 
> f1b8177 
>   
> contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java
>  718c5a5 
>   
> contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/ConnectionFactory.java
>  1442748 
>   
> contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java
>  6072d28 
>   
> contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/Services.java
>  1bace94 
> 
> Diff: https://reviews.apache.org/r/47306/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing.
> 
> 
> Thanks,
> 
> Gaurav Nagar
> 
>

Reply via email to