Re: Sentry repository is now relocated

r-authentication/ > > 3. You can use any mobile for token generation. I have used > "*Authy*" > >3. Wait about an hours time for things to sync up > >4. Great, you now have r/w access to both gitbox.apache.org < > >http://gitbox.apache.o

Re: Plan to migrate to gitbox.apache.org

/www.facebook.com/cloudera> [image: Cloudera > on LinkedIn] <https://www.linkedin.com/company/cloudera> > -- > > > On Mon, Jan 14, 2019 at 5:07 AM Colm O hEigeartaigh > wrote: > > > Hi Kalyan, > > > > I can update the Jenkins job - just let me know when the gitbox

Re: Plan to migrate to gitbox.apache.org

you force any other issues please respond back. > > Also, I would need some help in updating the Jenkins job( > https://builds.apache.org/job/PreCommit-SENTRY-Build). I'm still trying to > figure out a way to change the job to start using the new repository. > If anyone cou

Re: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

issue, and hope that your > > project can coordinate voluntarily moving your repositories in a > > timely manner. > > > > All settings, such as commit mail targets, issue linking, PR > > notification schemes etc will automatically be migrated to gitbox as > > well. > > > > With regards, Daniel on behalf of ASF Infra. > > > > PS:For inquiries, please reply to us...@infra.apache.org, not your > > project's dev list :-). > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Maven Artifacts

Thanks! On Wed, Oct 3, 2018 at 5:00 PM Na Li wrote: > Colm, > > I just clicked "release" for orgapachesentry-1015. > > Thanks, > > Lina > > On Wed, Oct 3, 2018 at 8:54 AM Colm O hEigeartaigh > wrote: > >> Hi, >> >> Don't forget t

Maven Artifacts

Hi, Don't forget to upload the maven artifacts for 2.1.0 as per: https://cwiki.apache.org/confluence/display/SENTRY/How+to+Release#HowtoRelease-PublishingApacheSentryartifactstomavencentral Colm. -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: VOTE] Release Sentry version 2.0.1

> > we are voting on the source:tag=release-2.0.1, SHA= > > a8954b221945154a43dd2a3180de909555310f85 > > (You can get the hash of the tag by doing "git rev-list release-2.0.1 | > > head -n 1" ) > > > > Vote will be open for 72 hours. > > > > [ ] +1 approve > > [ ] +0 no opinion > > [ ] -1 disapprove (and reason why) > > > > Thanks, > > Kalyan > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: svn commit: r28660 - in /release/sentry/2.0.1: ./ apache-sentry-2.0.1-bin.tar.gz apache-sentry-2.0.1-bin.tar.gz.asc apache-sentry-2.0.1-bin.tar.gz.sha apache-sentry-2.0.1-src.tar.gz apache-sentry-

; [image: Cloudera on Twitter] <https://twitter.com/cloudera> [image: > Cloudera on Facebook] <https://www.facebook.com/cloudera> [image: Cloudera > on LinkedIn] <https://www.linkedin.com/company/cloudera> > -- > > > On Mon, Aug 13,

Re: svn commit: r28660 - in /release/sentry/2.0.1: ./ apache-sentry-2.0.1-bin.tar.gz apache-sentry-2.0.1-bin.tar.gz.asc apache-sentry-2.0.1-bin.tar.gz.sha apache-sentry-2.0.1-src.tar.gz apache-sentry-

SIGNATURE- > > Added: release/sentry/2.0.1/apache-sentry-2.0.1-src.tar.gz.sha > > == > --- release/sentry/2.0.1/apache-sentry-2.0.1-src.tar.gz.sha (added) > +++ release/sentry/2.0.1/apache-sentry-2.0.1-src.tar.gz.sha Fri Aug 10 > 17:10:09 2018 > @@ -0,0 +1 @@ > +62f625a64da047a0d6db5a4cfb7b7d4713abdf5e apache-sentry-2.0.1-src.tar.gz > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Sentry-jdk-1.8 - Build # 110 - Still Failing

ache Jenkins build system has built Sentry-jdk-1.8 (build #110) > >> > >> Status: Still Failing > >> > >> Check console output at https://builds.apache.org/job/ > Sentry-jdk-1.8/110/ > >> to view the results. > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 67771: SENTRY-2285: Add a profile '-Pdist' to package Sentry binaries

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67771/#review205514 --- Ship it! Ship It! - Colm O hEigeartaigh On June 28, 2018, 2

Re: [DISCUSS] Annoying THIRD-PARTY.properties file

process except > when doing a release. Anyone with knowledge on how to avoid that file to be > modified? > > - Sergio > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66748/#review202127 --- Ship it! Ship It! - Colm O hEigeartaigh On April 27, 2018

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

> On April 25, 2018, 4:51 p.m., Colm O hEigeartaigh wrote: > > Why is the package name changed for GenericPrivilegeConverter.java + > > TSentryPrivilegeConverter.java? They've gone from > > "sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

it make more sense to leave them in the generic package? - Colm O hEigeartaigh On April 24, 2018, 7:29 p.m., Steve Moist wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > ht

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

> On April 23, 2018, 4:34 p.m., Colm O hEigeartaigh wrote: > > Shouldn't the location of the files be changed as well? If I apply the diff > > from this JIRA I see, for example: > > > > ./sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

> On April 23, 2018, 4:34 p.m., Colm O hEigeartaigh wrote: > > Shouldn't the location of the files be changed as well? If I apply the diff > > from this JIRA I see, for example: > > > > ./sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider

Re: Review Request 66748: Refactor out Sentry CLI from sentry-provider-db into own module

? - Colm O hEigeartaigh On April 20, 2018, 9:43 p.m., Steve Moist wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request 66452: SENTRY-2198 - Update to Kafka 1.0.0

distribution. Thanks, Colm O hEigeartaigh

Review Request 65923: SENTRY-2150 - Update Apache parent pom version

O hEigeartaigh

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

968325#file1968325line34> > > > > can you use "the hive-site.xml at hive server 2" instead of > > "hive-site.xml"? hive-site.xml can be at both hive server 2, HMS, or at > > sentry. > > Colm O hEigeartaigh wrote: > It's hiveserver2-site.xml t

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

ot;. Now the patch just has the correct tag to use. - Colm --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198544 --------

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198529 ------- On March 1, 2018, 12:32 p.m., Colm O hEigeartaigh wrote: > > -

Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

be "hive.security.authorization.manager". Diffs - sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAuthorizerFactory.java db8e4288 Diff: https://reviews.apache.org/r/65867/diff/1/ Testing --- Thanks, Colm O hEigeartaigh

Re: Review Request 65816: SENTRY-2145 - Some misc code cleanups

nding.java 908c80f1 sentry-core/sentry-core-model-solr/src/main/java/org/apache/sentry/core/model/solr/AdminOperation.java c054b7dc Diff: https://reviews.apache.org/r/65816/diff/2/ Changes: https://reviews.apache.org/r/65816/diff/1-2/ Testing --- Thanks, Colm O hEigeartaigh

Review Request 65816: SENTRY-2145 - Some misc code cleanups

iff: https://reviews.apache.org/r/65816/diff/1/ Testing --- Thanks, Colm O hEigeartaigh

Re: [DISCUSS] Allowing Java 8 features in Sentry

ict > > >> Java 7 compatibility, so we may just as well embrace the reality and > > start > > >> using Java 8 which hopefully will improve the quality/readability of > the > > >> code. > > >> > > >> Any thoughts on this? > > >> > > >> - Alex > > >> > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: permission request to edit sentry web page

> > Th widget in the page will pull it. > > > > -Kalyan > > > > -Kalyan > > > > On Fri, Jan 26, 2018 at 10:47 AM, Colm O hEigeartaigh < > cohei...@apache.org > > > > > wrote: > > > > > Hi Lina, > > > > >

Re: permission request to edit sentry web page

eas > > Thanks, > > Lina > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 65192: SENTRY-2074: Fix maven dependencies to have all directly used libraries defined

a third party dependency, as we will have to analyse which of its dependencies have also changed and modify the Sentry poms accordingly. - Colm O hEigeartaigh On Jan. 17, 2018, 8:42 p.m., Brian Towles wrote

Re: Issue with SimpleCacheProviderBackend

de, the column info is not set in ReadEntity if > HIVE_STATS_COLLECT_SCANCOLS is false. > > if (HiveConf.getBoolVar(this.conf, ConfVars.HIVE_STATS_COLLECT_SCANCOLS)) { > this.putAccessedColumnsToReadEntity(this.inputs, > this.columnAccessInfo); > } > > > Thanks, &

Re: Issue with SimpleCacheProviderBackend

r test code*, so I can >> >>see how you grant the privilege and therefore the cached privilege >> contains >> >>column? >> >> - I tried to use "GRANT *SELECT(*)* ON TABLE t1 TO ROLE >> >> user_role1", and got foll

[ANNOUNCE] Apache Sentry 1.7.1 released

-- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Issue with SimpleCacheProviderBackend

while >> compiling statement: FAILED: ParseException line 1:6 cannot recognize >> input >> near 'GRANT' 'SELECT' '(' in ddl statement >> - at org.apache.hive.service.cli.operation.Operation.toSQLExcepti >> on(Operation.java:380) >> - a

Re: [DISCUSS] Propose new Sentry PMC - Kalyan Kalvagadda

PMC team to help the project > being more active on important PMC discussions around issues and future > Sentry roadmap ideas. > > Given that, I'd like to propose that we vote to make Kalyan a new PMC on > the Apache Sentry project. > > - Sergio Pena > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: [DISCUSS] Propose new Sentry committer - Na Li

patches reviewed and committed more often. > > - Sergio Pena > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

1.7.1 JIRA

Hi Alex, Could you "release" 1.7.1 in JIRA please? Colm. -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: [VOTE] - Release Apache Sentry 1.7.1

gt;> Agree. I checked the rest of the release (checksums, gpg, build) and it is >> good. >> >> +1 >> >> On Wed, Dec 20, 2017 at 2:51 AM Colm O hEigeartaigh <cohei...@apache.org> >> wrote: >> >> > Yeah I saw that afterwards + updated the bra

Re: [VOTE] - Release Apache Sentry 1.7.1

need to add those? > > Sergio > > On Mon, Dec 18, 2017 at 8:58 AM, Colm O hEigeartaigh <cohei...@apache.org> > wrote: > >> This is a vote to release Apache Sentry 1.7.1. >> >> Artifacts: >> >> https://dist.apache.org/repos/dist/dev/sentry/1.7.1/

Re: Issue with SimpleCacheProviderBackend

ar Kalvagadda < kkal...@cloudera.com> wrote: > Colm, > > I will look closer into this today and see If i can help you out. > > -Kalyan > > On Mon, Dec 18, 2017 at 4:52 AM, Colm O hEigeartaigh <cohei...@apache.org> > wrote: > >> Hi, >> >> I've

[VOTE] - Release Apache Sentry 1.7.1

This is a vote to release Apache Sentry 1.7.1. Artifacts: https://dist.apache.org/repos/dist/dev/sentry/1.7.1/ Git tag: https://github.com/apache/sentry/tree/release-1.7.1 Issues fixed: https://issues.apache.org/jira/projects/SENTRY/versions/12342308 +1 from me. Colm. -- Colm O

Re: Issue with SimpleCacheProviderBackend

there are multiple > > issues with it and no one is actually using the fix. Anyone wants to do > it? > > > > - Alex > > > > On Wed, Dec 13, 2017 at 4:44 AM, Na Li <lina...@cloudera.com> wrote: > > > > > Colm, > > > > > > Glad you find th

Re: Issue with SimpleCacheProviderBackend

That would be great, thanks! Colm. On Tue, Dec 12, 2017 at 4:36 PM, Na Li <lina...@cloudera.com> wrote: > Colm, > > I suspect it is a bug in SENTRY-1291. I can take a look later today. > > Thanks, > > Lina > > On Tue, Dec 12, 2017 at 4:32 AM, Colm O hEigearta

Re: [RESULT][VOTE] Release Sentry version 2.0.0

Pena, Alexander Kolbasov, Colm o hEigeartaigh > > > Thanks to everyone who voted! I'll continue with the rest of the > release process. > > > -Kalyan > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Need permissions to update upstream wiki

What's your cwiki username though? Colm. On Wed, Dec 6, 2017 at 4:56 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Colm, > > My apache user name is just kalyan. > > -Kalyan > > On Wed, Dec 6, 2017 at 3:55 AM, Colm O hEigeartaigh <cohei...@apache.or

Re: [VOTE] Sentry Release 2.0.0 RC3

mmit;h= > > d77edb5056426dbf6106ee068ad5ae176fe0b225> > > (You can get the hash of the tag by doing "git rev-list release-2.0.0 | > > head -n 1" ) > > > > Vote will be open for a minimum 72 hours. > > > > [ ] +1 approve > > [ ] +0 no opinion > > [ ] -1 disapprove (and reason why) > > > > -Kalyan > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: [VOTE] Release Sentry 2.0.0 RC2

It's a "-1" from me unfortunately, as the source distribution does not build. I've submitted a patch here: https://issues.apache.org/jira/browse/SENTRY-2086 Colm. On Mon, Dec 4, 2017 at 2:30 PM, Colm O hEigeartaigh <cohei...@apache.org> wrote: > It's a lot easier IMO jus

Re: [VOTE] Release Sentry 2.0.0 RC2

gt; 1008 > > > > > > <https://repository.apache.org/content/repositories/ > > > > > orgapachesentry-1005/> > > > > > > > > > > > > > > > > > > Tag to be voted on > > > > > > *https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0 > > > > > > <https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0>* > > > > > > > > > > > > Sentry's KEYS containing the PGP key we used to sign the release: > > > > > > http://www.apache.org/dist/sentry/KEYS > > > > > > > > > > > > we are voting on the source:tag=release-2.0.0, SHA= > > > > > > c1ca5b22ba8a3a34398e0d62802cf65cfc47b4d1 > > > > > > (You can get the hash of the tag by doing "git rev-list > > > release-2.0.0 | > > > > > > head -n 1" ) > > > > > > > > > > > > Vote will be open for 72 hours. > > > > > > > > > > > > [ ] +1 approve > > > > > > [ ] +0 no opinion > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > > > -Kalyan > > > > > > > > > > > > > > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

he license. Same for the licenses above it. - Colm O hEigeartaigh On Dec. 1, 2017, 2:03 p.m., kalyan kumar kalvagadda wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revi

Re: [VOTE] Release Sentry version 2.0.0

to update the license file > in the repo? > > -Kalyan > > On Wed, Nov 29, 2017 at 1:52 PM, Colm O hEigeartaigh <cohei...@apache.org> > wrote: > >> Hi Kalyan, >> >> Anything Apache related you can immediately discount - so Hadoop, Hive, >> HBase,

Re: [VOTE] Release Sentry version 2.0.0

; > > org.apache.sentry.service.thrift.SentryService.call( > > > > SentryService.java:198) > > > > ... 5 more > > > > > > > > On Tue, Nov 28, 2017 at 4:38 PM, Kalyan Kumar Kalvagadda < > > > > kkal...@cloudera.com> wrote: > > > > > > > > > This is the release of Apache Sentry, version 2.0.0. > > > > > > > > > > It fixes the following issues: > > > > > *https://issues.apache.org/jira/projects/SENTRY/versions/12341081 > > > > > <https://issues.apache.org/jira/projects/SENTRY/versions/12341081 > >* > > > > > > > > > > Source and bin files : > > > > > *http://home.apache.org/~kalyan/apache-sentry-2.0.0-src-rc-1/ > > > > > <http://home.apache.org/~kalyan/apache-sentry-2.0.0-src-rc-1/>* > > > > > > > > > > Maven artifacts are available > > > > > here:https://repository.apache.org/content/ > > > repositories/orgapachesentry- > > > > > 1005/ > > > > > > > > > > > > > > > Tag to be voted on > > > > > *https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0 > > > > > <https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0>* > > > > > > > > > > Sentry's KEYS containing the PGP key we used to sign the release: > > > > > http://www.apache.org/dist/sentry/KEYS > > > > > > > > > > we are voting on the source:tag=release-2.0.0, SHA= > > > > > 18fe7c596fa1ffad3e656a42d534ac190876b642 > > > > > (You can get the hash of the tag by doing "git rev-list > > release-1.8.0 > > > | > > > > > head -n 1" ) > > > > > > > > > > Vote will be open for 72 hours. > > > > > > > > > > [ ] +1 approve > > > > > [ ] +0 no opinion > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > -Kalyan > > > > > > > > > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: [VOTE] Release Sentry version 2.0.0

; > > > SentryService.java:422) > > > > at > > > > org.apache.sentry.service.thrift.SentryService. > > > > runServer(SentryService.java:268) > > > > at > > > > org.apache.sentry.service.thrift.SentryService.call( > > > > SentryService.java:198) > > > > ... 5 more > > > > > > > > On Tue, Nov 28, 2017 at 4:38 PM, Kalyan Kumar Kalvagadda < > > > > kkal...@cloudera.com> wrote: > > > > > > > > > This is the release of Apache Sentry, version 2.0.0. > > > > > > > > > > It fixes the following issues: > > > > > *https://issues.apache.org/jira/projects/SENTRY/versions/12341081 > > > > > <https://issues.apache.org/jira/projects/SENTRY/versions/12341081 > >* > > > > > > > > > > Source and bin files : > > > > > *http://home.apache.org/~kalyan/apache-sentry-2.0.0-src-rc-1/ > > > > > <http://home.apache.org/~kalyan/apache-sentry-2.0.0-src-rc-1/>* > > > > > > > > > > Maven artifacts are available > > > > > here:https://repository.apache.org/content/ > > > repositories/orgapachesentry- > > > > > 1005/ > > > > > > > > > > > > > > > Tag to be voted on > > > > > *https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0 > > > > > <https://git-wip-us.apache.org/repos/asf/sentry/?p= > > > > > sentry.git;a=tag;h=refs/tags/release-2.0.0>* > > > > > > > > > > Sentry's KEYS containing the PGP key we used to sign the release: > > > > > http://www.apache.org/dist/sentry/KEYS > > > > > > > > > > we are voting on the source:tag=release-2.0.0, SHA= > > > > > 18fe7c596fa1ffad3e656a42d534ac190876b642 > > > > > (You can get the hash of the tag by doing "git rev-list > > release-1.8.0 > > > | > > > > > head -n 1" ) > > > > > > > > > > Vote will be open for 72 hours. > > > > > > > > > > [ ] +1 approve > > > > > [ ] +0 no opinion > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > -Kalyan > > > > > > > > > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: [VOTE] Release Sentry version 2.0.0

=release-2.0.0, SHA= > 18fe7c596fa1ffad3e656a42d534ac190876b642 > (You can get the hash of the tag by doing "git rev-list release-1.8.0 | > head -n 1" ) > > Vote will be open for 72 hours. > > [ ] +1 approve > [ ] +0 no opinion > [ ] -1 disapprove (and reason why) > > -Kalyan > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: 2.0 release and SENTRY-1812

Yes, it's already merged and the Jira is resolved. Colm. On 28 Nov 2017 7:04 p.m., "Alexander Kolbasov" wrote: Colm, do you plan to include SENTRY-1812 in 2.0 release?

Re: New release branch 2.0.0

*** >> >> >> >> > > I'm running below command. > >> mvn -s ~/.m2/settings.xml clean deploy -Psign-artifacts > > > I still observe the same issue. > > -Kalyan > > On Tue, Nov 28, 2017 at 10:46 AM, Colm O hEigeartaigh <cohei...@apache.org &g

Re: New release branch 2.0.0

Colm, the wiki mentioned "apache.staging.https". is that the correct one? > > On Tue, Nov 28, 2017 at 10:14 AM, Colm O hEigeartaigh <cohei...@apache.org > > wrote: > >> Have you got a "apache.releases.https" server defined in your maven >> settings.xml,

Re: New release branch 2.0.0

Btw, the branch should be named branch-2.0 as the Wiki suggests (git > >> > > checkout -b branch-1.8). It is better to name a branch this way so > we > >> can > >> > > commit bugfixes on it for future 2.0.x bugfix releases. > >> > > > >> > > On Mon, Nov 27, 2017 at 8:19 AM, Kalyan Kumar Kalvagadda < > >> > > kkal...@cloudera.com> wrote: > >> > > > >> > > > I've just created new release branch for upcoming 2.0.0 release. > >> Please > >> > > > continue committing to master branch as usual. > >> > > > I'll cherry-pick commits to branch-2.0.0 on need basis. > >> > > > > >> > > > -Kalyan > >> > > > > >> > > > >> > > >> > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 63874: SENTRY-1812 - Provide interactive Sentry CLI

: https://reviews.apache.org/r/63874/diff/2-3/ Testing --- Tested the CLI against the Sentry service. Thanks, Colm O hEigeartaigh

Re: HDFS tests failing

locally? > > On Fri, Nov 17, 2017 at 10:25 AM, Colm O hEigeartaigh <cohei...@apache.org > > > wrote: > > > Apologies if this is already captured in a JIRA, but some of the HDFS > tests > > are consistently failing for

Re: Cutting sentry 2.0.0 off master

wrote: > Colm/Mano Kovacs, > > Do you have an ETA on SENTRY-1812 and SENTRY-2012? > > -Kalyan > > -Kalyan > > On Fri, Nov 17, 2017 at 11:00 AM, Colm O hEigeartaigh <cohei...@apache.org > > wrote: > >> Fine with me, but I'd like to see the fixes for SE

Re: Cutting sentry 2.0.0 off master

er on the HMS plugin side > 2. SENTRY-2048 Bump Hive version to 2.3.2 > > If we cut the branch, we can limit commits to sentry 2.0.0 to that are > needed and isolate the root cause for the test failures and stabilize 2.0.0 > branch. > > -Kalyan > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

HDFS tests failing

expected:<1> but was:<0> [ERROR] Run 3: TestSentryHDFSServiceProcessor.testRequestSyncUpdatesWhenPubSubNotifyReturnsFullPathsUpdate:172 expected:<1> but was:<0> [ERROR] Run 4: TestSentryHDFSServiceProcessor.testRequestSyncUpdatesWhenPubSubNotifyReturnsFullPathsUpdate:172 expecte

Re: Review Request 63874: SENTRY-1812 - Provide interactive Sentry CLI

: https://reviews.apache.org/r/63874/diff/1-2/ Testing --- Tested the CLI against the Sentry service. Thanks, Colm O hEigeartaigh

Re: Review Request 63415: SENTRY-2012 Make SentryShellGeneric extendible

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63415/#review191314 --- Ship it! Ship It! - Colm O hEigeartaigh On Nov. 16, 2017, 6

Review Request 63874: SENTRY-1812 - Provide interactive Sentry CLI

. Thanks, Colm O hEigeartaigh

Re: Review Request 63668: SENTRY-2038 - Some ShellCommand improvements

7bafd8c4 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentryShellHive.java adfd102c Diff: https://reviews.apache.org/r/63668/diff/2/ Changes: https://reviews.apache.org/r/63668/diff/1-2/ Testing --- Tested the script. Thanks, Colm O hEigeartaigh

Re: Review Request 63668: SENTRY-2038 - Some ShellCommand improvements

sts the roles that are associated with all of the available groups. - Colm --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63668/#review190862 ------

Re: Review Request 63415: SENTRY-2012 Make SentryShellGeneric extendible

.lang.NullPointerException at org.apache.sentry.core.model.sqoop.SqoopModelAuthorizables.create(SqoopModelAuthorizables.java:31) Reading the privileges seems to work fine. - Colm O hEigeartaigh On Nov. 9, 2017, 3:32 p.m., Mano Kovacs wrote: > > --- > This is an automaticall

Review Request 63668: SENTRY-2038 - Some ShellCommand improvements

ellHive.java adfd102c Diff: https://reviews.apache.org/r/63668/diff/1/ Testing --- Tested the script. Thanks, Colm O hEigeartaigh

Review Request 63518: SENTRY-1997 - "Bump sqoop dependency version to 1.99.7"

0f0496bc Diff: https://reviews.apache.org/r/63518/diff/1/ Testing --- Tested successfully with a deployment as well. Thanks, Colm O hEigeartaigh

Re: Review Request 63303: SENTRY-1992 Improve parameter handling for SentryGenericProviderBackend

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63303/#review189415 --- Ship it! Ship It! - Colm O hEigeartaigh On Oct. 27, 2017

Re: Review Request 63185: SENTRY-2013 - Align the SentryGenericServiceClient and SentryPolicyServiceClient a bit more closely

y at all. - Colm --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63185/#review188994 --- On Oct. 25, 2017, 8:51 a.m., Colm O

Re: Review Request 63185: SENTRY-2013 - Align the SentryGenericServiceClient and SentryPolicyServiceClient a bit more closely

estBase.java b27dbc27 Diff: https://reviews.apache.org/r/63185/diff/2/ Changes: https://reviews.apache.org/r/63185/diff/1-2/ Testing --- Thanks, Colm O hEigeartaigh

Review Request 63215: SENTRY-2015 - Refactor Command implementations

/org/apache/sentry/provider/db/tools/command/hive/RevokeRoleFromGroupsCmd.java 86773ca4 Diff: https://reviews.apache.org/r/63215/diff/1/ Testing --- Tested the SentryShell with both Hive + Generic. Thanks, Colm O hEigeartaigh

Re: Review Request 63160: SENTRY-1667: Switching to Jetty v9 library

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63160/#review188909 --- Ship it! Ship It! - Colm O hEigeartaigh On Oct. 22, 2017, 3

Re: Review Request 63160: SENTRY-1667: Switching to Jetty v9 library

er-db/src/test/java/org/apache/sentry/provide r/db/service/thrift/TestSentryWebServerWithKerberos.java" intentional? - Colm O hEigeartaigh On Oct. 22, 2017, 3:10 a.m., Sergio Pena wrote: > > --- > This is an automatical

Review Request 63185: SENTRY-2013 - Align the SentryGenericServiceClient and SentryPolicyServiceClient a bit more closely

estBase.java b27dbc27 Diff: https://reviews.apache.org/r/63185/diff/1/ Testing --- Thanks, Colm O hEigeartaigh

Re: Review Request 63160: SENTRY-1667: Switching to Jetty v9 library

> On Oct. 19, 2017, 7:25 p.m., kalyan kumar kalvagadda wrote: > > I'm fine with bumping up the jetty version but the only concern is that the > > Solr tests will be failing unitl changes for SENTRY-1475 are committed. Agreed, probably best waiting until the Solr upgrade happens, assuming of

Review Request 63069: SENTRY-1996 - Rename Sqoop SentryAuthorizationHander

2687bc52 Diff: https://reviews.apache.org/r/63069/diff/1/ Testing --- Thanks, Colm O hEigeartaigh

Review Request 63026: SENTRY-1979 - Consolidate code for converting Hive privilege objects to Strings

/TestSentryShellHive.java 81059c5f Diff: https://reviews.apache.org/r/63026/diff/1/ Testing --- Tested the Hive Shell. Thanks, Colm O hEigeartaigh

Re: Proposal to release Sentry 2.0.0

t; > > > > > > > I have created a umbrella jira for releasing Sentry 2.0 . > > > > > > > > 1. SENTRY-1982 <https://issues.apache.org/jira/browse/SENTRY-1982> > > > -Release > > > > sentry 2.0.0 upstream > > > > > > > > > > > > I have linked issues that are blocking the release. If you see any > > issue > > > > that is blocking please attach it to this jira. That way we can fix > > them > > > > and clear all the road blocks for releasing SENTYR 2.0.0 > > > > > > What is your impression once you looked at these issues - do you think > > > that you should be able to fix majority of these or do you think that > > these > > > ca nbe simply moved out of the release? > > > > > Issues blocking SENTRY-1982 should be fixed before sentry 2.0.0 is > > released. > > > > > > > > > > > > > > > > > > > -Kalyan > > > > > > - Alex > > > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Do Sentry sync-up meetings exist?

ooms here and how >> to >> > > request an invitation? >> > > http://sentry.apache.org/community/get_involved.html >> > > >> > > On Wed, Oct 4, 2017 at 5:19 PM, Alexander Kolbasov < >> ak...@cloudera.com> >> > > wrote: >&g

Review Request 62812: SENTRY-1975 - Add sqoop support to SentryShellGeneric

/diff/1/ Testing --- Tested sentryShell with sqoop privileges. Thanks, Colm O hEigeartaigh

Re: Review Request 62791: SENTRY-1974 - Consolidate the Solr + Kafka PrivilegeValidators into a single GenericPrivilegeValidator

/TestSentryConfigToolSolr.java d199d20e Diff: https://reviews.apache.org/r/62791/diff/2/ Changes: https://reviews.apache.org/r/62791/diff/1-2/ Testing --- Tested Solr + Kafka privileges in the Shell. Thanks, Colm O hEigeartaigh

Review Request 62791: SENTRY-1974 - Consolidate the Solr + Kafka PrivilegeValidators into a single GenericPrivilegeValidator

://reviews.apache.org/r/62791/diff/1/ Testing --- Tested Solr + Kafka privileges in the Shell. Thanks, Colm O hEigeartaigh

Re: Code consolidation query

Name())); > privilege.setURI(fromNULLCol(mSentryPrivilege.getURI())); > if (mSentryPrivilege.getGrantOption() != null) { > privilege.setGrantOption(TSentryGrantOption.valueOf(mSentryPrivilege. > getGrantOption().toString().toUpperCase())); > } else { > privilege.setGrantOption(TSentryGrantOption.U

Re: Proposal to have sentry 1.9 release.

the release of sentry-HA which is > > well tested locally at cloudera. This feature is something that the > > community is waiting for quite some time. Why don't we release current > > master as sentry 1.9 release? > > > > Sentry 1.9.0 if released, would belo

Re: Do Sentry sync-up meetings exist?

> >> It is https://apache-sentry.slack.com <https://apache-sentry.slack.com/ > > > >> - I created room in my private slack account - sent Colm an invite. > >> > >> > >>> On Oct 4, 2017, at 1:36 AM, Colm O hEigeartaigh <cohei...@apache.org>

Re: Review Request 62767: SENTRY-1973 - Use KafkaPrivilegeModel to retrieve Kafka validators

/java/org/apache/sentry/provider/db/generic/tools/TestSentryShellKafka.java 183e83d3 Diff: https://reviews.apache.org/r/62767/diff/2/ Changes: https://reviews.apache.org/r/62767/diff/1-2/ Testing --- Tested the SentryShell works ok. Thanks, Colm O hEigeartaigh

Review Request 62767: SENTRY-1973 - Use KafkaPrivilegeModel to retrieve Kafka validators

/db/generic/tools/KafkaTSentryPrivilegeConverter.java 688bc9e4 Diff: https://reviews.apache.org/r/62767/diff/1/ Testing --- Tested the SentryShell works ok. Thanks, Colm O hEigeartaigh

Re: Do Sentry sync-up meetings exist?

e > > past? > > > > I've been in other Apache communities where they have these google chat > > sync-up meetings every 1 or 2 months to talk about what they'd like to do > > next in their products. Do you guys think this should be a reasonable > thing

Review Request 62745: SENTRY-1210 - Refactor the SentryShellSolr and SentryShellKafka

/generic/tools/TestSentryShellSolr.java d4e26e82 Diff: https://reviews.apache.org/r/62745/diff/1/ Testing --- Tested that the binary "sentryShell" can list privileges correctly for both Kafka and Solr. Thanks, Colm O hEigeartaigh

Re: Concerns about Hive authz2 support

shell commands, it is >> possible to bypass the client security checks. >> >> See https://cwiki.apache.org/confluence/display/Hive/SQL+Sta >> ndard+Based+Hive+Authorization >> >> The above problem is the reason Hive introduced a new authorization API >> ca

Code consolidation query

to remove CommandUtil altogether and just reference the methods in SentryServiceUtil. Should these extra pieces also be in SentryServiceUtil? Or is there a reason that they are different? Colm. -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Sentry CLI

> Solr/Kafka privilege model as well. Any volunteers? > > - Alex > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Concerns about Hive authz2 support

g if we remove any hive-authz2 profile and just add the > hive-authz2 classes to the current sentry-binding-hive module so that users > are allowed to switch either to v1 or v2 (for testing). Also for the tests, > find a way to run all sentry-tests-hive with v1 and v2 to validate the > qual

Re: Review Request 62631: SENTRY-1962: Merge hive 2.0 authz1/authz2 profile dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62631/#review186583 --- Ship it! Ship It! - Colm O hEigeartaigh On Sept. 27, 2017

Re: [DISCUSS] Allow JDK8 specific features on Sentry 2.0

eases are not active but companies are still > pretty > > > > active > > > > > > on Sentry. > > > > > > > > > > > > Questions to answer: > > > > > > - What should we do? > > > > > > - If we decide to keep JDK7 compatibility, how long should we > keep > > > this > > > > > > until we move completely to JDK8? > > > > > > > > > > > > In my opinion, companies will always be outdated with what we do > as > > > an > > > > > > Apache community. Taking a look at what we are doing with > SentryHA > > > > > > redesign, this is a breaking change for companies too because > fixes > > > on > > > > > this > > > > > > new design may not work for other companies and/or the backports > > > could > > > > be > > > > > > harder. > > > > > > > > > > > > However, the current Sentry 2.0 has been active with JDK7 as the > > > > support, > > > > > > and users active on 2.0 may be using JDK7 environments only. > > > > > > > > > > > > So, > > > > > > Should we wait until Sentry 2.1 or newer releases to allow JDK8 > > > > features? > > > > > > Should we start in Sentry 2.0? > > > > > > > > > > > > Any thoughts? > > > > > > > > > > > > - Sergio > > > > > > > > > > > > > > > > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Review Request 62411: SENTRY-1958: Bump to Hive version 2.0

.org/r/62411/ > --- > > (Updated Sept. 19, 2017, 3:10 p.m.) > > > Review request for sentry, Alexander Kolbasov, Colm O hEigeartaigh, and > kalyan kumar kalvagadda. > > > Bugs: sentry-1958 > https://issues.apache.org/jira/browse/sentry-1958 > > > Repository: sent

Re: [DISCUSSION] Move to JDK8 and Datanucleus 4

atanucleus 3 is also old, > and > > > other components already switched to version 4 as well. We already have > > > support for version 4, so the question here is if we should drop > > > Datanucleus 3 support and just use version 4 as the default. > > > > > > Sentry 2.0 is our current major version development, so it makes sense > to > > > do this move in this version. > > > > > > What do you all think? > > > > > > - Sergio > > > > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

  1   2   >