Re: gemfire-modules-session compile failures

2016-02-08 Thread Anthony Baker
I’m also hitting an error: * What went wrong: Execution failed for task ':extensions/gemfire-modules-assembly:install'. > Could not publish configuration 'archives' > Cannot publish artifact 'gemfire-modules-assembly.jar (org.apache.geode:extensions/gemfire-modules-assembly:1.0.0-incubating-SN

Re: gemfire-modules-session compile failures

2016-02-08 Thread Anthony Baker
Yep, this is GEODE-26. Anthony > On Feb 8, 2016, at 7:24 PM, John Blum wrote: > > mavenLocal() should definitely *not* be in the repositories { ... } > declaration > > [0] > of build.gradle. > > I am

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #215 was SUCCESSFUL (with 1244 tests)

2016-02-08 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #215 was successful. --- Scheduled 1248 tests in total. https://build.spring.io/browse/SGF-NAG-215/ -- This

Re: [ANNOUNCE] Apache Geode (incubating) release 1.0.0-incubating.M1

2016-02-08 Thread Nitin Lamba
And thanks for being my Commiter Quarterback! :) Kudos to everyone for making the first milestone - time to celebrate! Now for the next goal: M2 before Geode Summit!! Nitin From: Anthony Baker Sent: Monday, February 8, 2016 11:28 AM Cc: dev@geode.incubato

Re: gemfire-modules-session compile failures

2016-02-08 Thread John Blum
mavenLocal() should definitely *not* be in the repositories { ... } declaration [0] of build.gradle. I am actually quite surprised this passed either the Apache or Sonatype's release process. Sonatype

[GitHub] incubator-geode pull request: Addresses the documentation componen...

2016-02-08 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/43#issuecomment-181666323 Looks like these {@linkplain } links are causing javadoc warns when this is merged to the latest develop: ./gradlew gemfire-core:javadoc

Re: Pull Request Request

2016-02-08 Thread Dan Smith
I'll merge it. -Dan On Mon, Feb 8, 2016 at 5:41 PM, Kirk Lund wrote: > I'm going to have to abort this and let someone else handle it. The change > is so old I'm having trouble with pulling it into develop. > > -Kirk > > > On Mon, Feb 8, 2016 at 9:48 AM, Dave Barnes wrote: > > > Thanks, Kirk.

Re: Pull Request Request

2016-02-08 Thread Kirk Lund
I'm going to have to abort this and let someone else handle it. The change is so old I'm having trouble with pulling it into develop. -Kirk On Mon, Feb 8, 2016 at 9:48 AM, Dave Barnes wrote: > Thanks, Kirk. > > On Mon, Feb 8, 2016 at 9:31 AM, Kirk Lund wrote: > > > I'll take it. Sorry for the

Re: Review Request 43341: Fixing eclipse project generation for extensions

2016-02-08 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43341/#review118347 --- Ship it! Ship It! - Jens Deppe On Feb. 9, 2016, 12:34 a.m.,

Review Request 43351: GEODE-602: CI failure: OutOfOffHeapMemoryDUnitTest.testSimpleOutOfOffHeapMemoryMemberDisconnects

2016-02-08 Thread Swapnil Bawaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43351/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-08 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43346/#review118345 --- gemfire-core/src/main/java/com/gemstone/gemfire/admin/internal/Fi

Re: gemfire-modules-session compile failures

2016-02-08 Thread Anthony Baker
I think we should remove mavenLocal() from the repositories list. Anthony > On Feb 8, 2016, at 2:40 PM, Dan Smith wrote: > > Thanks, that worked! I think it's looking in ~/.m2 because we've added > mavenLocal as a maven repository in our gradle build. I must have at some > point sucked down j

Re: [ANNOUNCE] Apache Geode (incubating) release 1.0.0-incubating.M1

2016-02-08 Thread William Markito
Just updated the website to include the links to download. http://geode.incubator.apache.org/releases/ On Mon, Feb 8, 2016 at 11:28 AM Anthony Baker wrote: > Nitin, thanks for all your work as Release Manager! Great job! > > Anthony > > > > On Feb 8, 2016, at 10:47 AM, Nitin Lamba wrote: >

Re: Review Request 43341: Fixing eclipse project generation for extensions

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43341/ --- (Updated Feb. 9, 2016, 12:34 a.m.) Review request for geode, Anthony Baker, Jen

Re: Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43346/ --- (Updated Feb. 9, 2016, 12:33 a.m.) Review request for geode and Barry Oglesby.

Re: Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43346/ --- (Updated Feb. 9, 2016, 12:10 a.m.) Review request for geode and Barry Oglesby.

[GitHub] incubator-geode pull request: GEODE-909: CI failure: MemoryThresho...

2016-02-08 Thread sboorlagadda
Github user sboorlagadda commented on the pull request: https://github.com/apache/incubator-geode/pull/91#issuecomment-181629680 RBT url: https://reviews.apache.org/r/43345/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-geode pull request: GEODE-909: CI failure: MemoryThresho...

2016-02-08 Thread sboorlagadda
GitHub user sboorlagadda opened a pull request: https://github.com/apache/incubator-geode/pull/91 GEODE-909: CI failure: MemoryThresholdsOffHeapDUnitTest.testDisabledThresholds * Set cacheserver port to 'zero' in-order for system to pick-up random port instead of using AvailablePor

Re: Review Request 43345: GEODE-909: CI failure: MemoryThresholdsOffHeapDUnitTest.testDisabledThresholds

2016-02-08 Thread Swapnil Bawaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43345/#review118324 --- Ship it! Ship It! - Swapnil Bawaskar On Feb. 8, 2016, 11:22

Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43346/ --- Review request for geode and Barry Oglesby. Repository: geode Description ---

Review Request 43345: GEODE-909: CI failure: MemoryThresholdsOffHeapDUnitTest.testDisabledThresholds

2016-02-08 Thread Sai Boorlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43345/ --- Review request for geode and Swapnil Bawaskar. Repository: geode Description

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Kirk Lund
> On Feb. 8, 2016, 10:01 p.m., Dan Smith wrote: > > I don't think adding the jar to all projects is the correct way to fix > > this. We really shouldn't have *anything* that we're adding to the > > classpath of all projects unless it's really needed in all projects. > > Kirk Lund wrote: >

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Dan Smith
> On Feb. 8, 2016, 10:01 p.m., Dan Smith wrote: > > I don't think adding the jar to all projects is the correct way to fix > > this. We really shouldn't have *anything* that we're adding to the > > classpath of all projects unless it's really needed in all projects. > > Kirk Lund wrote: >

Re: Review Request 43341: Fixing eclipse project generation for extensions

2016-02-08 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43341/#review118310 --- Should this change be combined with this bit of logic? https://gi

Re: gemfire-modules-session compile failures

2016-02-08 Thread Dan Smith
Thanks, that worked! I think it's looking in ~/.m2 because we've added mavenLocal as a maven repository in our gradle build. I must have at some point sucked down jetty into my local maven repo. It seems like it still should have worked, but maybe there is a gradle bug here with a local maven repo

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Kirk Lund
> On Feb. 8, 2016, 9:53 p.m., Anthony Baker wrote: > > This dependency is already in gemfire-core/build.gradle. If it is needed > > in the root project can you remove it from gemfire-core? When I execute "./gradlew javadoc" with the changeset in the diff, it fixes the followign javadoc warnin

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Kirk Lund
> On Feb. 8, 2016, 10:01 p.m., Dan Smith wrote: > > I don't think adding the jar to all projects is the correct way to fix > > this. We really shouldn't have *anything* that we're adding to the > > classpath of all projects unless it's really needed in all projects. When I execute "./gradlew j

Re: gemfire-modules-session compile failures

2016-02-08 Thread Jens Deppe
I wonder why it's trying to resolve from your maven repo and not ~/.gradle? Can you try after deleting /home/dsmith/.m2/repository/org/eclipse/jetty/jetty-http/9.3.6.v20151106? --Jens On Mon, Feb 8, 2016 at 2:23 PM, Dan Smith wrote: > Anyone else hitting this after the merge of the modules? It

Review Request 43341: Fixing eclipse project generation for extensions

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43341/ --- Review request for geode, Anthony Baker, Jens Deppe, and Kirk Lund. Repository:

gemfire-modules-session compile failures

2016-02-08 Thread Dan Smith
Anyone else hitting this after the merge of the modules? It looks like it's trying to find a jetty-http-tests.jar and failing for some reason. This is with c8c26e6. -Dan >open> ./gradlew clean build ... What went wrong: Could not resolve all dependencies for configuration ':extensions/gemfire

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43338/#review118302 --- I don't think adding the jar to all projects is the correct way to

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43338/#review118300 --- This dependency is already in gemfire-core/build.gradle. If it is

Re: Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43338/ --- (Updated Feb. 8, 2016, 9:46 p.m.) Review request for geode and Dan Smith. Bug

Review Request 43338: GEODE-945: Add javax.transaction dependency to satisfy javadoc references

2016-02-08 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43338/ --- Review request for geode and Dan Smith. Bugs: GEODE-945 https://issues.apac

NOTICE/LICENSE files updates

2016-02-08 Thread Roman Shaposhnik
Hi! since there's been quite a bit of work around $subj lately, I wanted to share this link that has recently came up in a conversation on general@incubator: http://creadur.apache.org/whisker/ Thanks, Roman.

New javadoc warnings

2016-02-08 Thread Kirk Lund
Someone introduced new javadoc warnings. I'm not sure which commit caused this. :pivotalgf-assembly:docsC:\dev\gemfire_CLEAN\open\gemfire-core\src\main\java\com\gemstone\gemfire\cache\FailedSynchronizationException.java:33: warning - Tag @see: reference not found: javax.transaction.UserTransaction

Re: [ANNOUNCE] Apache Geode (incubating) release 1.0.0-incubating.M1

2016-02-08 Thread Anthony Baker
Nitin, thanks for all your work as Release Manager! Great job! Anthony > On Feb 8, 2016, at 10:47 AM, Nitin Lamba wrote: > > The Apache Geode team is proud to announce the first incubation release > version ‘1.0.0-incubating.M1’ of Apache Geode. > > Apache Geode (incubating) is a data manag

[ANNOUNCE] Apache Geode (incubating) release 1.0.0-incubating.M1

2016-02-08 Thread Nitin Lamba
The Apache Geode team is proud to announce the first incubation release version ‘1.0.0-incubating.M1’ of Apache Geode. Apache Geode (incubating) is a data management platform that provides a database-like consistency model, reliable transaction processing and a shared-nothing architecture to ma

Re: 1.0.0-incubating.M1

2016-02-08 Thread Roman Shaposhnik
First of all -- congrats on your first release -- this is a pretty big milestone! On Mon, Feb 8, 2016 at 9:36 AM, Kirk Lund wrote: > Is a 1.0.0-incubating.M1 release something that we can advertise outside of > the dev@geode list, blog about, etc? Yes, absolutely! Make sure to update the website

Re: Review Request 43265: GEODE-826 replaced gradlew with bigtop script

2016-02-08 Thread Hitesh Khamesra
> On Feb. 6, 2016, 12:30 a.m., Dan Smith wrote: > > Looks good, but it's still lame that we have to do this at all. We're > > losing support for windows without cygwin and moving away from the standard > > gradle tooling. > > > > Did you test this with an without a fresh checkout to excerise t

Re: Pull Request Request

2016-02-08 Thread Dave Barnes
Thanks, Kirk. On Mon, Feb 8, 2016 at 9:31 AM, Kirk Lund wrote: > I'll take it. Sorry for the delay Dave! > > -Kirk > > > On Sat, Feb 6, 2016 at 10:51 AM, Dave Barnes wrote: > > > Would some conscientious committer be so kind as to accept PR #43? It's a > > simple Javadoc update that's been in t

[GitHub] incubator-geode pull request: Addresses the documentation componen...

2016-02-08 Thread kirklund
Github user kirklund commented on the pull request: https://github.com/apache/incubator-geode/pull/43#issuecomment-181492310 I'll take this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

Re: Review Request 43265: GEODE-826 replaced gradlew with bigtop script

2016-02-08 Thread Kirk Lund
> On Feb. 6, 2016, 6:25 p.m., Jens Deppe wrote: > > -1 for no Windows support. Yeah this is a concern. I'm still doing quite a bit of building on Windows. Is there a way to somehow support both? Maybe at a minimum we could add a page to the wiki that describes how a Windows developer could mod

Re: 1.0.0-incubating.M1

2016-02-08 Thread Kirk Lund
Is a 1.0.0-incubating.M1 release something that we can advertise outside of the dev@geode list, blog about, etc? -Kirk On Mon, Feb 8, 2016 at 9:34 AM, Anthony Baker wrote: > Hi, > > After a successful IPMC VOTE [1] on the 1.0.0-incubating.M1 I believe we > have completed the release finalizati

1.0.0-incubating.M1

2016-02-08 Thread Anthony Baker
Hi, After a successful IPMC VOTE [1] on the 1.0.0-incubating.M1 I believe we have completed the release finalization steps. Nitin is planning to send the ANNOUNCE email as noted in [2]. The release bits have propagated to the mirrors and Maven Central [3]. The release branch has been merged

Re: Pull Request Request

2016-02-08 Thread Kirk Lund
I'll take it. Sorry for the delay Dave! -Kirk On Sat, Feb 6, 2016 at 10:51 AM, Dave Barnes wrote: > Would some conscientious committer be so kind as to accept PR #43? It's a > simple Javadoc update that's been in the queue since November. > Thanks in advance, > -Dave >

Build failed in Jenkins: Geode-nightly #371

2016-02-08 Thread Apache Jenkins Server
See Changes: [abaker] Fix version whitespace -- [...truncated 323 lines...] :gemfire-pulse:sourcesJar :gemfire-pulse:war :gemfire-pulse:signArchives SKIPPED :gemfire-wan:jar :gemfire-wan:javadoc :ge