RE: [Discussion] Refactor the sentry-provider

2016-05-19 Thread Ma, Junjie
Hi Sravya, Thanks for the comments. For the AuthorizationProvider, Policy, ProviderBackend, the relationship should be AuthorizationProvider depend on Policy, Policy depend on ProviderBackend. Currently, AuthorizationProvider and ProviderBackend in one module, it makes problem to the dependency

Re: [Discussion] Refactor the sentry-provider

2016-05-19 Thread Sravya Tirukkovalur
Hi Colin, Thanks for initiating this discussion. I think some refactoring in sentry-provider definitely helps. - At a high level, we should at the least break it down into a client module and a server module, so that sentry clients do not have to depend on server jar for example. - There are some

[Discussion] Refactor the sentry-provider

2016-05-19 Thread Ma, Junjie
Hi all, For the sentry-provider, it will make person confusion because many isolated concepts are included, eg, authorization provider, provider backend, sentry service(server&client). I suggest to refactor the structure of sentry-provider, for example, split into 3 parts as following: sentry-

Re: Review Request 47630: SENTYR-1272:Enable ALTERVIEW_RENAME in hive binding

2016-05-19 Thread Dapeng Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47630/#review134084 --- sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/te

Review Request 47630: SENTYR-1272:Enable ALTERVIEW_RENAME in hive binding

2016-05-19 Thread Ke Jia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47630/ --- Review request for sentry, Colin Ma and Dapeng Sun. Repository: sentry Descri

Re: Review Request 47628: SENTRY-1274: Enable ALTERVIEW_AS in hive binding

2016-05-19 Thread Dapeng Sun
> On 五月 20, 2016, 10:35 a.m., Dapeng Sun wrote: > > Ship It! LGTM, better to fill the field "Bugs" with jira id likes "SENTRY-1274" next time. - Dapeng --- This is an automatically generated e-mail. To reply, visit: https://reviews.apa

Re: Review Request 47628: SENTRY-1274: Enable ALTERVIEW_AS in hive binding

2016-05-19 Thread Dapeng Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47628/#review134081 --- Ship it! Ship It! - Dapeng Sun On 五月 20, 2016, 10:22 a.m., K

Review Request 47628: SENTRY-1274: Enable ALTERVIEW_AS in hive binding

2016-05-19 Thread Ke Jia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47628/ --- Review request for sentry, Colin Ma and Dapeng Sun. Repository: sentry Descri

Re: Review Request 47569: SENTRY-1265: Sentry service should not require a TGT as it is not talking to other kerberos services as a client

2016-05-19 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47569/ --- (Updated May 20, 2016, 12:08 a.m.) Review request for sentry, Hao Hao and Lenni

Sentry-jdk-1.7 - Build # 665 - Still Failing

2016-05-19 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #665) Status: Still Failing Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/665/ to view the results.