Re: Review Request 39445: GEODE-12/GEODE-304 Fix Pulse tests

2015-10-27 Thread Nitin Lamba
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39445/ --- (Updated Oct. 28, 2015, 4:29 a.m.) Review request for geode, Dick Cavender, Tus

Re: Review Request 39445: GEODE-12/GEODE-304 Fix Pulse tests (WIP)

2015-10-27 Thread Nitin Lamba
> On Oct. 19, 2015, 7:14 p.m., Mark Bretl wrote: > > pulse/build.gradle, line 43 > > > > > > gemfire-core is using version 4.12. Can the version be upgraded from > > 4.10 to 4.12? > > Nitin Lamba wrote: > I'm no

Re: Review Request 39445: GEODE-12/GEODE-304 Fix Pulse tests (WIP)

2015-10-27 Thread Nitin Lamba
> On Oct. 27, 2015, 10:43 a.m., Tushar Khairnar wrote: > > pulse/src/main/java/com/vmware/gemfire/tools/pulse/internal/data/JMXDataUpdater.java, > > line 2350 > > > > > > We can Remove gemXD realated code completel

[GitHub] incubator-geode pull request: Fixes GEODE-495.

2015-10-27 Thread sboorlagadda
Github user sboorlagadda closed the pull request at: https://github.com/apache/incubator-geode/pull/25 --- 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 f

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #108 has FAILED

2015-10-27 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #108 failed. --- Scheduled No failed tests found, a possible compilation error. https://build.spring.io/browse/SG

Re: Review Request 39706: GEODE-411: fix suspect string on disconnect

2015-10-27 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39706/#review104227 --- Ship it! Ship It! - Dan Smith On Oct. 27, 2015, 11:31 p.m., Dar

Review Request 39706: GEODE-411: fix suspect string on disconnect

2015-10-27 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39706/ --- Review request for geode and Dan Smith. Bugs: GEODE-411 https://issues.apac

Jenkins build is back to normal : Geode-nightly #259

2015-10-27 Thread Apache Jenkins Server
See

Re: Review Request 39699: GEODE-192: Removing TransactionFunctionService

2015-10-27 Thread Swapnil Bawaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39699/#review104222 --- Ship it! Ship It! - Swapnil Bawaskar On Oct. 27, 2015, 8:44 p.m

[GitHub] incubator-geode pull request: Fixes GEODE-495.

2015-10-27 Thread sboorlagadda
GitHub user sboorlagadda opened a pull request: https://github.com/apache/incubator-geode/pull/25 Fixes GEODE-495. To use a random port for the JMX manager, also fixed "cleanup" not to create cache if not already created in-order to delete a region. You can merge this pull request

Re: Review Request 39605: GEODE-77: Coordinator shutdown does not trigger coordinator reassignment

2015-10-27 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39605/#review104213 --- Ship it! I have a suggesting for your tests. The product changes

Re: Review Request 39605: GEODE-77: Coordinator shutdown does not trigger coordinator reassignment

2015-10-27 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39605/ --- (Updated Oct. 27, 2015, 9:41 p.m.) Review request for geode, anilkumar gingade,

Re: Review Request 39605: GEODE-77: Coordinator shutdown does not trigger coordinator reassignment

2015-10-27 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39605/ --- (Updated Oct. 27, 2015, 9:25 p.m.) Review request for geode, anilkumar gingade,

Review Request 39699: GEODE-192: Removing TransactionFunctionService

2015-10-27 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39699/ --- Review request for geode and Swapnil Bawaskar. Repository: geode Description

Re: Propose changing check to only depend on UnitTest

2015-10-27 Thread Mark Bretl
I have checked in GEODE-487 into develop to remove integrationTests and distributedTests from the build task. The new task for running all tests is 'precheckin'. I have also updated the nightly build with this change. --Mark On Thu, Oct 1, 2015 at 5:30 PM, Nitin Lamba wrote: > +1 . > > Initial

Re: Bump language level to 1.8

2015-10-27 Thread William Markito
Yes, but just because the base image we created for Docker is already JDK 8 based. On Tue, Oct 27, 2015 at 9:41 AM, Anthony Baker wrote: > If we the jenkins build in a docker container that would fix this issue > right? GEODE-60? > > Anthony > > > On Oct 27, 2015, at 9:01 AM, Dan Smith wrote:

Re: Bump language level to 1.8

2015-10-27 Thread Anthony Baker
If we the jenkins build in a docker container that would fix this issue right? GEODE-60? Anthony > On Oct 27, 2015, at 9:01 AM, Dan Smith wrote: > > Looks like the apache jenkins server is currently running java 1.7. Can > someone with access to it fix this? > > https://builds.apache.org/job

Re: Bump language level to 1.8

2015-10-27 Thread William Markito
Just saw it and fixed. Let's see the next build. Thanks! On Tue, Oct 27, 2015 at 9:01 AM, Dan Smith wrote: > Looks like the apache jenkins server is currently running java 1.7. Can > someone with access to it fix this? > > https://builds.apache.org/job/Geode-nightly/258/console > > * What went

Re: Bump language level to 1.8

2015-10-27 Thread Dan Smith
Looks like the apache jenkins server is currently running java 1.7. Can someone with access to it fix this? https://builds.apache.org/job/Geode-nightly/258/console * What went wrong: Execution failed for task ':gemfire-junit:compileTestJava'. > invalid source release: 1.8 On Mon, Oct 26, 2015

Re: Review Request 39585: Integrated Security DUnitTest

2015-10-27 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39585/#review104173 --- We need to have you remove usage of hydra.Log. All hydra classes ar

Re: Review Request 39445: GEODE-12/GEODE-304 Fix Pulse tests (WIP)

2015-10-27 Thread Tushar Khairnar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39445/#review104160 --- pulse/src/main/java/com/vmware/gemfire/tools/pulse/internal/data/J

[GitHub] incubator-geode pull request: Wrong one line build command

2015-10-27 Thread jaoki
GitHub user jaoki opened a pull request: https://github.com/apache/incubator-geode/pull/24 Wrong one line build command How to build Geode in Spark Connector document seems odd. It should be 2 lines but it is combined into one line. You can merge this pull request into a Git rep