Re: Question about user-level privileges

2018-01-30 Thread Sergio Pena
See inline comments. On Mon, Jan 29, 2018 at 4:37 PM, Alexander Kolbasov wrote: > I am wondering what is the relationship between "users" as defined in > Sentry and users as defined in Unix or LDAP or Active Directory. > > Should it be allowed to assign permissions to a user that doesn't exist?

Re: Review Request 65244: SENTRY-2074: Fix maven dependencies to have all directly used libraries defined

2018-01-30 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65244/#review196467 --- pom.xml Lines 1299 (patched)

Re: How to run bin/run_sentry.sh

2018-01-30 Thread Xinran Yu Tinney
Hi, Sasha, what do you mean change ownership of files? Also, Lina, I have run the command but it seems something removed the JAVA_HOME, please see the attachment. 2018-01-12 16:57 GMT-06:00 Alexander Kolbasov : > Running it via studio is not the best idea since it will change ownership > of files

Re: Review Request 65268: SENTRY-1904: TransactionManager should limit the max time spent by transaction retry

2018-01-30 Thread kalyan kumar kalvagadda via Review Board
> On Jan. 30, 2018, 1:38 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionManager.java > > Lines 249 (patched) > > > > > >

Re: Review Request 65268: SENTRY-1904: TransactionManager should limit the max time spent by transaction retry

2018-01-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65268/ --- (Updated Jan. 31, 2018, 12:30 a.m.) Review request for sentry, Arjun Mishra, Na

Re: How to run bin/run_sentry.sh

2018-01-30 Thread Alexander Kolbasov
The way run_sentry works - it actually runs mvn exec:java which builds code in your local repo and creates jar files. If you run this as root, these files will be owned by root which is not good, so please do not run this via sudo to save you some trouble. A simple way to use it to run sentry: ru