Re: Review Request 69353: SENTRY-2454: Add new sentry store api to gather the privileges for a list of authorizables.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69353/#review212150 --- Ship it! Ship It! - Na Li On Dec. 18, 2018, 11:23 p.m., kalyan kumar kalvagadda wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69353/ > --- > > (Updated Dec. 18, 2018, 11:23 p.m.) > > > Review request for sentry and Sergio Pena. > > > Bugs: SENTRY-2454 > https://issues.apache.org/jira/browse/SENTRY-2454 > > > Repository: sentry > > > Description > --- > > New sentry API should be implemented to fetch the privileges granted to > authorizables and it's children. authorizables include database, tables, > columns and URI's. > > > Diffs > - > > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > 63f53752f5a376015dce642ca1cb59aaa1dd16ba > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java > 85ea6d1c06c84f89108fb1313f505dba5e324eb3 > > sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java > b327e9e510483787311bf5218eac4039f04291ff > > > Diff: https://reviews.apache.org/r/69353/diff/4/ > > > Testing > --- > > Added new unit tests to test the API added. > > > Thanks, > > kalyan kumar kalvagadda > >
Re: Review Request 69353: SENTRY-2454: Add new sentry store api to gather the privileges for a list of authorizables.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69353/ --- (Updated Dec. 18, 2018, 11:23 p.m.) Review request for sentry and Sergio Pena. Bugs: SENTRY-2454 https://issues.apache.org/jira/browse/SENTRY-2454 Repository: sentry Description --- New sentry API should be implemented to fetch the privileges granted to authorizables and it's children. authorizables include database, tables, columns and URI's. Diffs (updated) - sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 63f53752f5a376015dce642ca1cb59aaa1dd16ba sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java 85ea6d1c06c84f89108fb1313f505dba5e324eb3 sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java b327e9e510483787311bf5218eac4039f04291ff Diff: https://reviews.apache.org/r/69353/diff/4/ Changes: https://reviews.apache.org/r/69353/diff/3-4/ Testing --- Added new unit tests to test the API added. Thanks, kalyan kumar kalvagadda
Re: Review Request 69353: SENTRY-2454: Add new sentry store api to gather the privileges for a list of authorizables.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69353/ --- (Updated Dec. 11, 2018, 12:28 a.m.) Review request for sentry and Sergio Pena. Changes --- Addressed review comments. Summary (updated) - SENTRY-2454: Add new sentry store api to gather the privileges for a list of authorizables. Bugs: SENTRY-2454 https://issues.apache.org/jira/browse/SENTRY-2454 Repository: sentry Description --- New sentry API should be implemented to fetch the privileges granted to authorizables and it's children. authorizables include database, tables, columns and URI's. Diffs (updated) - sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java e2d6c85ac6b285e286c12f7eec669b841cf76e9d sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java e48eea377b842475f72b6fab4567a82c8fd93098 sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java ca8c41610d3dcd12b267579c3d3cbe5b7ba7b208 Diff: https://reviews.apache.org/r/69353/diff/3/ Changes: https://reviews.apache.org/r/69353/diff/2-3/ Testing --- Added new unit tests to test the API added. Thanks, kalyan kumar kalvagadda