Re: Review Request 47306: Ambari Admin Privilege required for Pig and Hive View. 403 error received when opening Pig View by a non-admin user

2016-05-17 Thread DIPAYAN BHOWMICK

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



Re: Review Request 47306: Ambari Admin Privilege required for Pig and Hive View. 403 error received when opening Pig View by a non-admin user

2016-05-16 Thread Nitiraj Rathore

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


Ship it!




Ship It!

- Nitiraj Rathore


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



Review Request 47306: Ambari Admin Privilege required for Pig and Hive View. 403 error received when opening Pig View by a non-admin user

2016-05-12 Thread Gaurav Nagar

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

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