Re: Sentry repository is now relocated

2019-01-20 Thread Colm O hEigeartaigh
-two-factor-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://gitbo

Re: Plan to migrate to gitbox.apache.org

2019-01-18 Thread Colm O hEigeartaigh
lt;https://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

Re: Plan to migrate to gitbox.apache.org

2019-01-14 Thread Colm O hEigeartaigh
t; If 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

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

2018-12-13 Thread Colm O hEigeartaigh
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

2018-10-04 Thread Colm O hEigeartaigh
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 for

Maven Artifacts

2018-10-03 Thread Colm O hEigeartaigh
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

2018-08-16 Thread Colm O hEigeartaigh
> > > 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-

2018-08-13 Thread Colm O hEigeartaigh
; [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, 201

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-

2018-08-13 Thread Colm O hEigeartaigh
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

2018-07-27 Thread Colm O hEigeartaigh
.scm.SCM.checkout(SCM.java:504) > at hudson.model.AbstractProject.checkout(AbstractProject.java:1208) > at > hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:574) > at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)

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

2018-07-26 Thread Colm O hEigeartaigh
>> The Apache 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

2018-06-28 Thread Colm O hEigeartaigh
--- 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

2018-06-28 Thread Colm O hEigeartaigh
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

2018-04-30 Thread Colm O hEigeartaigh
--- 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

2018-04-26 Thread Colm O hEigeartaigh
> 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/

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

2018-04-25 Thread Colm O hEigeartaigh
o wouldn't 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,

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

2018-04-24 Thread Colm O hEigeartaigh
> 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/pro

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

2018-04-24 Thread Colm O hEigeartaigh
> 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/pro

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

2018-04-23 Thread Colm O hEigeartaigh
tools? - 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.a

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

2018-04-04 Thread Colm O hEigeartaigh
distribution. Thanks, Colm O hEigeartaigh

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

2018-03-06 Thread Colm O hEigeartaigh
O hEigeartaigh

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

2018-03-05 Thread Colm O hEigeartaigh
/ Changes: https://reviews.apache.org/r/65867/diff/1-2/ Testing --- Thanks, Colm O hEigeartaigh

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

2018-03-05 Thread Colm O hEigeartaigh
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

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

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

2018-03-02 Thread Colm O hEigeartaigh
--- 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

2018-03-01 Thread Colm O hEigeartaigh
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

2018-02-28 Thread Colm O hEigeartaigh
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

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

Re: [DISCUSS] Allowing Java 8 features in Sentry

2018-02-23 Thread Colm O hEigeartaigh
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

2018-01-27 Thread Colm O hEigeartaigh
; 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, > > > > > > W

Re: permission request to edit sentry web page

2018-01-26 Thread Colm O hEigeartaigh
Hi Lina, What's your cwiki username? Colm. On Fri, Jan 26, 2018 at 4:23 PM, Na Li wrote: > Hello, > > Can I have permission to edit the following page for sentry? > > https://cwiki.apache.org/confluence/display/SENTRY/ > Sentry+Roadmap+and+ideas > > Tha

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

2018-01-23 Thread Colm O hEigeartaigh
-2.2.2.jar jackson-jaxrs-1.8.3.jar jackson-mapper-asl-1.8.8.jar jackson-xc-1.8.3.jar - Colm O hEigeartaigh On Jan. 22, 2018, 3:12 p.m., Brian Towles wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

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

2018-01-19 Thread Colm O hEigeartaigh
ate 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: Review Request 65053: SENTRY-2120: Potential cross-site scripting in LogLevelServlet

2018-01-12 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65053/#review195308 --- Ship it! Ship It! - Colm O hEigeartaigh On Jan. 11, 2018

Re: Issue with SimpleCacheProviderBackend

2018-01-05 Thread Colm O hEigeartaigh
t 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, > > Lin

Re: Issue with SimpleCacheProviderBackend

2018-01-05 Thread Colm O hEigeartaigh
rivilege and therefore the cached privilege >> contains >> >>column? >> >> - I tried to use "GRANT *SELECT(*)* ON TABLE t1 TO ROLE >> >> user_role1", and got following error >> >> - >> >> - 2018-01-03 23:23:50

[ANNOUNCE] Apache Sentry 1.7.1 released

2018-01-05 Thread Colm O hEigeartaigh
-- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Issue with SimpleCacheProviderBackend

2018-01-05 Thread Colm O hEigeartaigh
li.operation.SQLOperation.prepare( >> SQLOperation.java:206) >> - at org.apache.hive.service.cli.operation.SQLOperation.runIntern >> al(SQLOperation.java:290) >> - at org.apache.hive.service.cli.operation.Operation.run(Operatio >> n

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

2018-01-04 Thread Colm O hEigeartaigh
eing 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

2018-01-04 Thread Colm O hEigeartaigh
d committed more often. > > - Sergio Pena > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

1.7.1 JIRA

2017-12-24 Thread Colm O hEigeartaigh
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

2017-12-24 Thread Colm O hEigeartaigh
ums, gpg, build) and it is >> good. >> >> +1 >> >> On Wed, Dec 20, 2017 at 2:51 AM Colm O hEigeartaigh >> wrote: >> >> > Yeah I saw that afterwards + updated the branch. I didn't think it was >> > worthy of cancelling the vote thoug

Re: [VOTE] - Release Apache Sentry 1.7.1

2017-12-20 Thread Colm O hEigeartaigh
Sergio > > On Mon, Dec 18, 2017 at 8:58 AM, Colm O hEigeartaigh > wrote: > >> 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: >> >>

Re: Issue with SimpleCacheProviderBackend

2017-12-18 Thread Colm O hEigeartaigh
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 > wrote: > >> Hi, >> >> I've done some further analy

[VOTE] - Release Apache Sentry 1.7.1

2017-12-18 Thread Colm O hEigeartaigh
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

2017-12-18 Thread Colm O hEigeartaigh
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 wrote: > > > > > Colm, > > > > > > Glad you find the cause! > > > > > > You can revert Sent

1.7.1 version in JIRA

2017-12-13 Thread Colm O hEigeartaigh
Can someone add a 1.7.1 version in JIRA please? Colm. -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Issue with SimpleCacheProviderBackend

2017-12-13 Thread Colm O hEigeartaigh
n add a e2e test case and set break point at where the authorization > object hierarchy to a list of authorization objects, which is used to do > exact match with cache > > Sent from my iPhone > > > On Dec 12, 2017, at 11:27 AM, Colm O hEigeartaigh > wrote: > > > &g

Re: Issue with SimpleCacheProviderBackend

2017-12-12 Thread Colm O hEigeartaigh
That would be great, thanks! Colm. On Tue, Dec 12, 2017 at 4:36 PM, Na Li 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 hEigeartaigh > wrote: > >

Issue with SimpleCacheProviderBackend

2017-12-12 Thread Colm O hEigeartaigh
able hierarchy looks like: Server [name=server1] Database [name=authz] Table [name=words] There is no "column" here, and a match is not made against the cached privilege as a result. Is this a bug or am I missing some configuration switch? Colm. -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

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

2017-12-08 Thread Colm O hEigeartaigh
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

2017-12-06 Thread Colm O hEigeartaigh
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 > wrote: &g

Re: Need permissions to update upstream wiki

2017-12-06 Thread Colm O hEigeartaigh
changes to upstream wiki page. > https://cwiki.apache.org/confluence/display/SENTRY/How+to+Release but I do > not have permissions to do it. > It would be great if any one who is authorized to grant me permission can > help me with that. > > > -Kalyan > --

Re: [VOTE] Sentry Release 2.0.0 RC3

2017-12-05 Thread Colm O hEigeartaigh
6106ee068ad5ae176fe0b225> > > (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

2017-12-04 Thread Colm O hEigeartaigh
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 wrote: > It's a lot easier IMO just letting the M

Re: [VOTE] Release Sentry 2.0.0 RC2

2017-12-04 Thread Colm O hEigeartaigh
ository.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 64259: SENTRY-1572 SentryMain() shouldn't dynamically load tool class

2017-12-04 Thread Colm O hEigeartaigh
f this fix? - Colm O hEigeartaigh On Dec. 1, 2017, 9:15 p.m., Xinran Tinney wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revi

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

2017-12-01 Thread Colm O hEigeartaigh
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

2017-11-30 Thread Colm O hEigeartaigh
d be using to update the license file > in the repo? > > -Kalyan > > On Wed, Nov 29, 2017 at 1:52 PM, Colm O hEigeartaigh > wrote: > >> Hi Kalyan, >> >> Anything Apache related you can immediately discount - so Hadoop, Hive, >> HBase, Solr, Zookeeper, etc.

Re: [VOTE] Release Sentry version 2.0.0

2017-11-29 Thread Colm O hEigeartaigh
pache.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

2017-11-29 Thread Colm O hEigeartaigh
; 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

2017-11-29 Thread Colm O hEigeartaigh
e. >> >> I'm not sure if these are blockers. I will wait for other to respond on >> this. >> >> While I'm waiting, I will start working on the duplicate jar dependency >> issue. >> >> >> -Kalyan >> >> On Wed, Nov 29, 2017 at 6

Re: [VOTE] Release Sentry version 2.0.0

2017-11-29 Thread Colm O hEigeartaigh
rce: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: 2.0 release and SENTRY-1812

2017-11-28 Thread Colm O hEigeartaigh
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

2017-11-28 Thread Colm O hEigeartaigh
; * >> >> >> >> > > 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 > w

Re: New release branch 2.0.0

2017-11-28 Thread Colm O hEigeartaigh
.staging.https". is that the correct one? > > On Tue, Nov 28, 2017 at 10:14 AM, Colm O hEigeartaigh > wrote: > >> Have you got a "apache.releases.https" server defined in your maven >> settings.xml, with a corresponding username + password? >> &g

Re: New release branch 2.0.0

2017-11-28 Thread Colm O hEigeartaigh
ranch 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

2017-11-20 Thread Colm O hEigeartaigh
anges: 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

2017-11-17 Thread Colm O hEigeartaigh
> [ERROR] Run 3: > > TestSentryHDFSServiceProcessor.testRequestSyncUpdatesWhenPubS > > ubNotifyReturnsFullPathsUpdate:172 > > expected:<1> but was:<0> > > [ERROR] Run 4: > > TestSentryHDFSServiceProcessor.testRequestSyncUpdatesWhenPubS > > ubNotifyRet

Re: Cutting sentry 2.0.0 off master

2017-11-17 Thread Colm O hEigeartaigh
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 > wrote: > >> Fine with me, but I'd like to see the fixes for SENTRY-1812 and >>

Re: Cutting sentry 2.0.0 off master

2017-11-17 Thread Colm O hEigeartaigh
rrier 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

2017-11-17 Thread Colm O hEigeartaigh
cted:<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

2017-11-17 Thread Colm O hEigeartaigh
anges: 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

2017-11-17 Thread Colm O hEigeartaigh
--- 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

2017-11-16 Thread Colm O hEigeartaigh
rvice. Thanks, Colm O hEigeartaigh

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

2017-11-15 Thread Colm O hEigeartaigh
Sqoop.java 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

2017-11-14 Thread Colm O hEigeartaigh
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

2017-11-10 Thread Colm O hEigeartaigh
ointerException 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

2017-11-08 Thread Colm O hEigeartaigh
tryShellHive.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"

2017-11-02 Thread Colm O hEigeartaigh
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

2017-10-27 Thread Colm O hEigeartaigh
--- 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 63303: SENTRY-1992 Improve parameter handling for SentryGenericProviderBackend

2017-10-26 Thread Colm O hEigeartaigh
atic final String COMPONENT_TYPE= "sentry.provider.backend.generic.component-type"; Apart from that it looks fine to me. - Colm O hEigeartaigh On Oct. 25, 2017, 7:01 p.m., Mano Kovacs wrote: > > --- > This is an autom

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

2017-10-25 Thread Colm O hEigeartaigh
e functionality 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:5

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

2017-10-25 Thread Colm O hEigeartaigh
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

2017-10-23 Thread Colm O hEigeartaigh
/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

2017-10-22 Thread Colm O hEigeartaigh
--- 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

2017-10-22 Thread Colm O hEigeartaigh
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 automatically g

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

2017-10-20 Thread Colm O hEigeartaigh
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

2017-10-20 Thread Colm O hEigeartaigh
> 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 co

Review Request 63069: SENTRY-1996 - Rename Sqoop SentryAuthorizationHander

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

Re: Proposal to release Sentry 2.0.0

2017-10-16 Thread Colm O hEigeartaigh
de for Solr 6, but he > will try to see if Solr 7 is easy to do, but the initial expectations were > to have Solr 6. > > What do you think about the plans? Any comments regarding about them? Would > you like to see something else or different? > > Sergio. > > On Mon, Oct

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

2017-10-16 Thread Colm O hEigeartaigh
/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

2017-10-16 Thread Colm O hEigeartaigh
tAnalyze schematic hooks. > > > I think you are referring to some planned follow-up work to actually > > solve > > > the authorization problem for Hive 2 - right? Yes. > > > > > > > > > > > > > > > > > > > I have created a

Re: Do Sentry sync-up meetings exist?

2017-10-16 Thread Colm O hEigeartaigh
I believe you can sign up here: https://the-asf.slack.com/ Colm. On Thu, Oct 5, 2017 at 8:51 PM, Alexander Kolbasov wrote: > Yes, moving to the official ASF slack team would make a lot of sense. What > should we do to move there? > > > On Thu, Oct 5, 2017 at 1:48 AM, Colm

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

2017-10-06 Thread Colm O hEigeartaigh
/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

2017-10-06 Thread Colm O hEigeartaigh
/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

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

Re: Code consolidation query

2017-10-05 Thread Colm O hEigeartaigh
ryPrivilege.getURI())); > if (mSentryPrivilege.getGrantOption() != null) { > privilege.setGrantOption(TSentryGrantOption.valueOf(mSentryPrivilege. > getGrantOption().toString().toUpperCase())); > } else { > privilege.setGrantOption(TSentryGrantOption.UNSET); > } > }

Re: Proposal to have sentry 1.9 release.

2017-10-05 Thread Colm O hEigeartaigh
ich 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 below features. > > > >1. Sentry-HA > >2. Integration with Hive 2.0 using auth-V1. > > > > Seeking inputs from every one in this regard. > > > > > > -Kalyan > > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com

Re: Do Sentry sync-up meetings exist?

2017-10-05 Thread Colm O hEigeartaigh
k.com/ > > > >> - I created room in my private slack account - sent Colm an invite. > >> > >> > >>> On Oct 4, 2017, at 1:36 AM, Colm O hEigeartaigh > >> wrote: > >>> > >>> Where is the Slack room? I don't see it in

  1   2   3   >