Re: Review Request 69212: SENTRY-2329: Integrate sentry with Hadoop 3.1.1
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69212/ --- (Updated Nov. 5, 2018, 8 p.m.) Review request for sentry and kalyan kumar kalvagadda. Bugs: sentry-2329 https://issues.apache.org/jira/browse/sentry-2329 Repository: sentry Description (updated) --- Bump hadoop.version to 3.1.1. This integration is compatible with old Hadoop 2.x versions. You can compile with Hadoop 2.x by changing the pom.xml or you can add the Sentry/HDFS binding jars built with Hadoop3 in the Hadoop 2 classpath. Both ways are verified and working. Diffs - pom.xml acbdcc2722bf189811cb528ac1b2d07983a571c2 sentry-binding/sentry-binding-solr/pom.xml f2a5fca76d3d220fcf2b72a3179ff5218fc6577c sentry-hdfs/sentry-hdfs-common/pom.xml df6f04c048b502ff5f8e8ec397d75166faba8c3c sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java c9ecc4021b167d98c7dade409c97ae7d26e967ea sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryINodeAttributesProvider.java 18b62652a11dfee6683cb8f24944ccd3d344dc9f sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithKerberos.java 5d94d4bc6a2a47189e69556a5e4d9bdee05952a7 Diff: https://reviews.apache.org/r/69212/diff/2/ Testing --- Thanks, Sergio Pena
Re: Review Request 69212: SENTRY-2329: Integrate sentry with Hadoop 3.1.1
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69212/#review210333 --- Ship it! Ship It! - kalyan kumar kalvagadda On Nov. 2, 2018, 6:40 p.m., Sergio Pena wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69212/ > --- > > (Updated Nov. 2, 2018, 6:40 p.m.) > > > Review request for sentry and kalyan kumar kalvagadda. > > > Bugs: sentry-2329 > https://issues.apache.org/jira/browse/sentry-2329 > > > Repository: sentry > > > Description > --- > > Bump hadoop.version to 3.1.1. The changes on Sentry are compatible with > Hadoop 2.7. > > > Diffs > - > > pom.xml acbdcc2722bf189811cb528ac1b2d07983a571c2 > sentry-binding/sentry-binding-solr/pom.xml > f2a5fca76d3d220fcf2b72a3179ff5218fc6577c > sentry-hdfs/sentry-hdfs-common/pom.xml > df6f04c048b502ff5f8e8ec397d75166faba8c3c > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java > c9ecc4021b167d98c7dade409c97ae7d26e967ea > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryINodeAttributesProvider.java > 18b62652a11dfee6683cb8f24944ccd3d344dc9f > > sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithKerberos.java > 5d94d4bc6a2a47189e69556a5e4d9bdee05952a7 > > > Diff: https://reviews.apache.org/r/69212/diff/2/ > > > Testing > --- > > > Thanks, > > Sergio Pena > >
Re: Review Request 69212: SENTRY-2329: Integrate sentry with Hadoop 3.1.1
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69212/#review210331 --- Sergio, I remeber we taking about supporting multiple versions of Hadoop somehow but this change is not backward compatible. People using sentry 2.2 are forced to use Hadoop 3.1.1 only. Is that correct? - kalyan kumar kalvagadda On Nov. 2, 2018, 6:40 p.m., Sergio Pena wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69212/ > --- > > (Updated Nov. 2, 2018, 6:40 p.m.) > > > Review request for sentry and kalyan kumar kalvagadda. > > > Bugs: sentry-2329 > https://issues.apache.org/jira/browse/sentry-2329 > > > Repository: sentry > > > Description > --- > > Bump hadoop.version to 3.1.1. The changes on Sentry are compatible with > Hadoop 2.7. > > > Diffs > - > > pom.xml acbdcc2722bf189811cb528ac1b2d07983a571c2 > sentry-binding/sentry-binding-solr/pom.xml > f2a5fca76d3d220fcf2b72a3179ff5218fc6577c > sentry-hdfs/sentry-hdfs-common/pom.xml > df6f04c048b502ff5f8e8ec397d75166faba8c3c > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java > c9ecc4021b167d98c7dade409c97ae7d26e967ea > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryINodeAttributesProvider.java > 18b62652a11dfee6683cb8f24944ccd3d344dc9f > > sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithKerberos.java > 5d94d4bc6a2a47189e69556a5e4d9bdee05952a7 > > > Diff: https://reviews.apache.org/r/69212/diff/2/ > > > Testing > --- > > > Thanks, > > Sergio Pena > >
Re: sentry1.5API
If you need to use sentry for table authorization I can help on building a binding Xinran > On Nov 5, 2018, at 10:55 AM, Kalyan Kumar Kalvagadda > wrote: > > Andy, > > I'm not sure what you are referring to but I want to make sure of thing. > Sentry doesn't have an a public API. You need a sentry-binding. > It would be helpfull if you could help us understand your use case and how > you are trying to use sentry. > >> On Mon, Nov 5, 2018 at 10:42 AM Na Li wrote: >> >> Andy, >> >> I don't think there is up-to-date document on Sentry API. >> >> You can find the API provided by Sentry server at >> >> https://github.com/apache/sentry/blob/master/sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/SentryPolicyService.java >> >> On how to integrate with your application, you can look at how Hive >> integrate with Sentry at >> >> https://github.com/apache/sentry/blob/master/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java >> >> You can run e2e test at >> >> https://github.com/apache/sentry/blob/master/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java#L112 >> and see how hive uses Sentry for authorization. >> >> Thanks, >> >> Lina >> >>> On Mon, Nov 5, 2018 at 8:51 AM andy chen wrote: >>> >>> Hello, >>> >>>Can you give me a favor? I want to ask where to find the >> document >>> of sentry API 。I try to read the source code of sentry,however ,I can’t >> get >>> the point . Now I stil don’t know how to call the sentry API in my app. >> I >>> really need you help. >>> >>> Thanks Regards >>> >>> >>> >>> >>> >>
Re: sentry1.5API
Andy, I don't think there is up-to-date document on Sentry API. You can find the API provided by Sentry server at https://github.com/apache/sentry/blob/master/sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/SentryPolicyService.java On how to integrate with your application, you can look at how Hive integrate with Sentry at https://github.com/apache/sentry/blob/master/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java You can run e2e test at https://github.com/apache/sentry/blob/master/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java#L112 and see how hive uses Sentry for authorization. Thanks, Lina On Mon, Nov 5, 2018 at 8:51 AM andy chen wrote: > Hello, > > Can you give me a favor? I want to ask where to find the document > of sentry API 。I try to read the source code of sentry,however ,I can’t get > the point . Now I stil don’t know how to call the sentry API in my app. I > really need you help. > > Thanks Regards > > > > >
sentry1.5API
Hello, Can you give me a favor? I want to ask where to find the document of sentry API 。I try to read the source code of sentry,however ,I can’t get the point . Now I stil don’t know how to call the sentry API in my app. I really need you help. Thanks Regards
