Build failed in Jenkins: Geode-nightly #364

2016-02-01 Thread Apache Jenkins Server
See Changes: [jcbledsoe] Fix footer GitHub link [ukohlmeyer] GEODE-795: Changed RegionEntryFactory code from complex nested if-else -- [...truncated 398 lines...] :gemfire-joptsimple:sourcesJar :g

Re: Build failed in Jenkins: Geode-nightly #359

2016-02-01 Thread Nitin Lamba
Seems the slaves still have the properties file lurking around so RAT has failed in the past few builds. The following file needs to be deleted from all slaves: /home/jenkins/jenkins-slave/workspace/Geode-nightly/gemfire-pulse/src/main/resources/pulseversion.properties Nitin Sent from Outlook

Re: [DISCUSS] Next Release Scope - version 1.0.0-incubating.M2

2016-02-01 Thread Michael Stolz
I'd like to see the focus on docs and dependencies first. Then incorporation of WAN and CQ features. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Fri, Jan 29, 2016 at 12:26 PM, Nitin Lamba wrote: > Thought of starting a separate thread to discuss next releas

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-02-01 Thread pdxrunner
Github user pdxrunner commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-178041809 I'm closing this PR in favor of looking at the longer term solution of gradlew vs. gradle in the release distros. The secondary problem I found (path error in

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-02-01 Thread pdxrunner
Github user pdxrunner closed the pull request at: https://github.com/apache/incubator-geode/pull/82 --- 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 this feature enabled and wishes so, or if the feat

Re: [VOTE] RC2: Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

2016-02-01 Thread Anthony Baker
Sounds good, and thanks for the feedback. It’s really helpful to get these reports of test failures from a broader range of systems than ASF Jenkins. Anthony > On Jan 31, 2016, at 9:36 PM, Brian Dunlap wrote: > > Thanks Anthony - > > To potentially help with tricky sleuthing... > I ran into

Re: Review Request 42991: GEODE-805: Fix javadoc warnings

2016-02-01 Thread Kirk Lund
> On Jan. 30, 2016, 6:43 p.m., Anthony Baker wrote: > > gemfire-joptsimple/src/main/java/joptsimple/internal/AbbreviationMap.java, > > line 57 > > > > > > Any reason to remove this link? http://www.perldoc.com/ and

Re: Broken javadocs

2016-02-01 Thread Kirk Lund
When I remove this line: diff --git a/build.gradle b/build.gradle index 13afa17..c2c5e40 100755 --- a/build.gradle +++ b/build.gradle @@ -312,7 +312,6 @@ subprojects { javadoc.classpath += configurations.provided javadoc { -options.addStringOption('Xdoclint:none', '-quiet') options

Re: Review Request 42941: Fix for GEODE-877 CacheXmlGenerator creates index xml with internal alias with expression.

2016-02-01 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42941/#review117228 --- Ship it! Ship It! - Barry Oglesby On Jan. 29, 2016, 2:09 a.m

Re: Build failed in Jenkins: Geode-nightly #359

2016-02-01 Thread Mark Bretl
I don't have karma/permissions to delete workspace on slaves, I can only delete them from Jenkins unfortunately. There is a Jenkins plugin which can delete the workspace before the build, however, the ASF Jenkins does not have it. I think the best workaround I can think of is to delete any untrack

Re: Review Request 42977: GEODE-875: CI failure: ParallelGatewaySenderOperationsOffHeapDUnitTest.testParallelGatewaySender_SingleNode_UserPR_localDestroy_SimultenuousPut_RecreateRegion

2016-02-01 Thread Barry Oglesby
> On Jan. 29, 2016, 10:46 p.m., xiaojian zhou wrote: > > gemfire-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/concurrent/ConcurrentParallelGatewaySenderOperation_2_DUnitTest.java, > > line 631 > > > > > >

Re: Broken javadocs

2016-02-01 Thread Dan Smith
For GEODE-518, I think we should just generate javadocs for the public API and not worry about the internal classes right now. That might make this a lot easier if we do that fix. -Dan On Mon, Feb 1, 2016 at 9:26 AM, Kirk Lund wrote: > When I remove this line: > > diff --git a/build.gradle b/bu

[GitHub] incubator-geode pull request: GEODE-887: correct paths in README.m...

2016-02-01 Thread pdxrunner
GitHub user pdxrunner opened a pull request: https://github.com/apache/incubator-geode/pull/86 GEODE-887: correct paths in README.md Corrected classpath shown for compiling and running app in Geode in 5 Minutes section of README.md You can merge this pull request into a Git reposit

Re: Broken javadocs

2016-02-01 Thread Anilkumar Gingade
Agree With Dan... -Anil. On Mon, Feb 1, 2016 at 10:20 AM, Dan Smith wrote: > For GEODE-518, I think we should just generate javadocs for the public API > and not worry about the internal classes right now. That might make this a > lot easier if we do that fix. > > -Dan > > On Mon, Feb 1, 2016

Re: Review Request 42991: GEODE-805: Fix javadoc warnings

2016-02-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42991/ --- (Updated Feb. 1, 2016, 6:38 p.m.) Review request for geode. Changes ---

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

2016-02-01 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42655/ --- (Updated Feb. 1, 2016, 6:56 p.m.) Review request for geode, Anthony Baker and D

Re: Broken javadocs

2016-02-01 Thread Kirk Lund
It's not really an issue of internal vs User API... I just merged feature/GEODE-805 into develop. The build is now currently free of any and all javadoc warnings. Here's the problem: If someone now commits a class with a malformed javadoc, the build output will start to collect new javadoc warnin

Re: Broken javadocs

2016-02-01 Thread Anthony Baker
The Java8 javadoc tool went crazy on strict compliance. The doc page shows that we could disable certain categories (e.g. missing) instead of all checks. http://docs.oracle.com/javase/8/docs/technotes/tools/windows/javadoc.html Anthony > On Feb 1, 2016, at 11:27 AM, Kirk Lund wrote: > > It's

Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43059/ --- Review request for geode, Bruce Schuchardt and Udo Kohlmeyer. Repository: geode

Re: [VOTE] RC2: Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

2016-02-01 Thread Anthony Baker
Hi Nitin, Is the next step to announce the result and start a vote on the general incubator list? Anthony > On Jan 27, 2016, at 12:21 PM, Nitin Lamba wrote: > > All, > > This is the second release candidate of the first release for Apache Geode, > version 1.0.0-incubating.M1. Thanks to all

Review Request 43062: GEODE-730 Optimize single filter join queries

2016-02-01 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43062/ --- Review request for geode, anilkumar gingade, Barry Oglesby, Dan Smith, and xiaoj

Review Request 43063: GEODE-631 CI failure: PRQueryRegionDestroyedDUnitTest.testPRWithRegionDestroyInOneDatastoreWithDelay

2016-02-01 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43063/ --- Review request for geode, anilkumar gingade, Barry Oglesby, Dan Smith, and xiaoj

Re: Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43059/#review117270 --- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/inter

Review Request 43071: GEODE-445: CI failure: OutOfOffHeapMemoryDUnitTest.testOtherMembersSeeOutOfOffHeapMemoryMemberDisconnects

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

Re: Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43059/#review117302 --- Ship it! Ship It! - Udo Kohlmeyer On Feb. 1, 2016, 8:27 p.m.

[GitHub] incubator-geode pull request: GEODE-646: OffHeapRegionEntryHelper ...

2016-02-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/83 --- 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 this feature enabled and wishes so, or if the feature

Re: Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Hitesh Khamesra
> On Feb. 1, 2016, 9:56 p.m., Bruce Schuchardt wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java, > > line 2273 > > > > > > I think this sho

Re: Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43059/ --- (Updated Feb. 2, 2016, 12:11 a.m.) Review request for geode, Bruce Schuchardt a

Re: Review Request 43059: Updated ViewCreator logic and fixed couple of issues

2016-02-01 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43059/#review117318 --- gemfire-core/src/test/java/com/gemstone/gemfire/distributed/inter

Review Request 43077: GEODE-799: Update Spring to 4.2.2 and SDG to 1.7.2

2016-02-01 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43077/ --- Review request for geode, Jinmei Liao and Kirk Lund. Repository: geode Descri

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

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

Review Request 43084: GEODE-892: Cleanup unnecessary compression code from wan test

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