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
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
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-
---
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
---
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
> 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
---
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
---
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
---
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
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.
10 matches
Mail list logo