Re: Review Request 56931: [2/2] SENTRY-1635: Limit HMS connections only to the leader of the sentry servers

2017-02-22 Thread Alexander Kolbasov

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


Ship it!




Ship It!

- Alexander Kolbasov


On Feb. 22, 2017, 4:08 p.m., Vamsee Yarlagadda wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56931/
> ---
> 
> (Updated Feb. 22, 2017, 4:08 p.m.)
> 
> 
> Review request for sentry, Alexander Kolbasov, Hao Hao, and kalyan kumar 
> kalvagadda.
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> Addressing the review comments made my Sasha in the code review: 
> https://reviews.apache.org/r/56892/
> 
> 
> Diffs
> -
> 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java
>  1a99a045cdcb7225ff29b379497ddbc8e5ec4c2b 
> 
> Diff: https://reviews.apache.org/r/56931/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Vamsee Yarlagadda
> 
>



Review Request 56931: [2/2] SENTRY-1635: Limit HMS connections only to the leader of the sentry servers

2017-02-22 Thread Vamsee Yarlagadda

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

Review request for sentry, Alexander Kolbasov, Hao Hao, and kalyan kumar 
kalvagadda.


Repository: sentry


Description
---

Addressing the review comments made my Sasha in the code review: 
https://reviews.apache.org/r/56892/


Diffs
-

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java
 1a99a045cdcb7225ff29b379497ddbc8e5ec4c2b 

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


Testing
---


Thanks,

Vamsee Yarlagadda