You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by kalyan kumar kalvagadda via Review Board <no...@reviews.apache.org> on 2020/07/23 23:39:00 UTC

Review Request 72706: SENTRY-2557: Queries are running too slow after when there are more than 4k roles

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

Review request for sentry and Na Li.


Repository: sentry


Description
-------

When there are more than 4k roles Hive queries are very slow below because of the time it spends on the compile phase because of the delay in fetching the permissions.


Diffs
-----

  sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 980c8ad6 


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


Testing
-------


Thanks,

kalyan kumar kalvagadda


Re: Review Request 72706: SENTRY-2557: Queries are running too slow after when there are more than 4k roles

Posted by Na Li via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72706/#review221345
-----------------------------------------------------------


Ship it!




Ship It!

- Na Li


On July 23, 2020, 11:38 p.m., kalyan kumar kalvagadda wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72706/
> -----------------------------------------------------------
> 
> (Updated July 23, 2020, 11:38 p.m.)
> 
> 
> Review request for sentry and Na Li.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> When there are more than 4k roles Hive queries are very slow below because of the time it spends on the compile phase because of the delay in fetching the permissions.
> 
> 
> Diffs
> -----
> 
>   sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 980c8ad6 
> 
> 
> Diff: https://reviews.apache.org/r/72706/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> kalyan kumar kalvagadda
> 
>