Re: Review Request 56709: SENTRY-1569 Added changes to sql scripts persist Perm/Path change Adding missing files

2017-02-22 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56709/ --- (Updated Feb. 22, 2017, 11:40 p.m.) Review request for sentry, Alexander

Re: Review Request 56709: SENTRY-1569 Added changes to sql scripts persist Perm/Path change Adding missing files

2017-02-22 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56709/ --- (Updated Feb. 22, 2017, 11:38 p.m.) Review request for sentry, Alexander

Re: Review Request 56954: SENTRY-1639 Added code changes to refactor the usage of configuration constants related to transport

2017-02-22 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56954/ --- (Updated Feb. 22, 2017, 9:53 p.m.) Review request for sentry, Alexander

Review Request 56954: SENTRY-1639 Added code changes to refactor the usage of configuration constants related to transport

2017-02-22 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56954/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, Mat Crocker, Vamsee

Review Request 56952: SENTRY-1637. Periodically purge Delta change tables.

2017-02-22 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/ --- Review request for sentry. Bugs: SENTRY-1637

Re: Review Request 54947: SENTRY-1556 Simplify privilege cleaning

2017-02-22 Thread kalyan kumar kalvagadda
> On Jan. 22, 2017, 8:39 a.m., Alexander Kolbasov wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 595 > > > > > > Can it

Re: Review Request 54947: SENTRY-1556 Simplify privilege cleaning

2017-02-22 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54947/#review165734 --- - kalyan kumar kalvagadda On Feb. 16, 2017, 5:04 p.m., kalyan

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,

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

Re: Review Request 56892: SENTRY-1635: Limit HMS connections only to the leader of the sentry servers

2017-02-22 Thread Vamsee Yarlagadda
> On Feb. 22, 2017, 1:48 a.m., Alexander Kolbasov wrote: > > Posted a new review for these changes: https://reviews.apache.org/r/56931/ - Vamsee --- This is an automatically generated e-mail. To reply, visit: