Re: Review Request 66146: SENTRY-2184: Performance Issue: MPath is queried for each MAuthzPathsMapping in full snapshot

2018-03-19 Thread Brian Towles via Review Board

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


Ship it!




Ship It!

- Brian Towles


On March 19, 2018, 1:43 p.m., Na Li wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66146/
> ---
> 
> (Updated March 19, 2018, 1:43 p.m.)
> 
> 
> Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar 
> kalvagadda, and Sergio Pena.
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> Configure DataNucleus to fetch MPath in bulk to improve performance
> 
> 
> Diffs
> -
> 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
>  d883c51 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  4521ad4 
> 
> 
> Diff: https://reviews.apache.org/r/66146/diff/1/
> 
> 
> Testing
> ---
> 
> Run unti tests.
> Check log message that veries that the query for MPath is issued in bullk.
> 
> 
> Thanks,
> 
> Na Li
> 
>



Review Request 66146: SENTRY-2184: Performance Issue: MPath is queried for each MAuthzPathsMapping in full snapshot

2018-03-19 Thread Na Li via Review Board

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

Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar 
kalvagadda, and Sergio Pena.


Repository: sentry


Description
---

Configure DataNucleus to fetch MPath in bulk to improve performance


Diffs
-

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
 d883c51 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 4521ad4 


Diff: https://reviews.apache.org/r/66146/diff/1/


Testing
---

Run unti tests.
Check log message that veries that the query for MPath is issued in bullk.


Thanks,

Na Li