Build failed in Jenkins: Geode-nightly #361

See Changes: [rvs] WAN and CQ code drop under the Pivotal SGA [upthewaterspout] GEODE-802: Add cq, wan projects to the gradle build [upthewaterspout] GEODE-804: Remaining WAN and CQ changes from pivotal [upthewaterspout] GEODE-803: Upda

Fwd: Confusion over NOTICE vs LICENSE files

FYI here's a good clarification on what must be done in LICENSE and NOTICE files when you add a dependency or copy source code into the GEODE project. Note that the linked doc hasn't passed legal review yet (but it's really helpful if you've never looked into this subject before). Anthony Beg

Re: OQL Group by on PDX objects

I think the PDX comparable conversation you remember was about Enums. When Enums are serialized by PDX we remember their ordinal value so they can be compared later (Enums are Comparable). But the PdxInstance used to represent a non-enum domain class is not Comparable. On Thu, Jan 28, 2016 at 10

WAN distributedTest failures

The WAN distributedTest failures were due to a change in locator communications that has now been backed out.

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

This build got an OOME parsing the test results - but I think it was due to the WAN test failures Bruce emailed about, so it should be resolved now. -Dan On Fri, Jan 29, 2016 at 4:35 AM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See

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

GitHub user pdxrunner opened a pull request: https://github.com/apache/incubator-geode/pull/82 Fixes GEODE-874 Replaced gradlew commands with gradle in text files. Corrected classpath shown for compiling and running app in Geode in 5 Minutes section of README.md You can merg

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

Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-176894749 You changed these to be ./gradle. I don't think that makes sense. Either someone is using the gradle wrapper - './gradlew' - or they are using in install

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

Github user pdxrunner commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-176913037 You're correct. I should have changed "./gradlew" to just "gradle" for the .txt and .md files I updated. I also agree that gradlew, not gradle, is the correct

Re: OQL Group by on PDX objects

Looks like you are trying to do group-by at top level object (Pdx); can you organize the fields so that one of the fields can be used for group-by...will that work for your application... Or you can do group-by on multiple fields of the object. E.g. select pickup_cell.x, pickup_cell.y group by (p

Re: Review Request 42884: GEODE-646: OffHeapRegionEntryHelper needs unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42884/#review116976 --- gemfire-core/src/main/java/com/gemstone/gemfire/internal/offheap/

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

Thought of starting a separate thread to discuss next release scope. So far we have the following suggestions: Dan: - GEODE-823 - remove gemfire-joptsimple, gemfire-json, rename to geode-, other misc issues - GEODE-818 - clean up dependencies - GEODE-572 - generate public javadocs (part of GEODE-

Re: Review Request 42884: GEODE-646: OffHeapRegionEntryHelper needs unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42884/ --- (Updated Jan. 29, 2016, 9:30 p.m.) Review request for geode, Darrel Schneider a

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42977/ --- Review request for geode, Dan Smith and xiaojian zhou. Bugs: GEODE-875 http

Re: Review Request 42884: GEODE-646: OffHeapRegionEntryHelper needs unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42884/#review117031 --- Ship it! Ship It! - Darrel Schneider On Jan. 29, 2016, 1:30

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

GitHub user sboorlagadda opened a pull request: https://github.com/apache/incubator-geode/pull/83 GEODE-646: OffHeapRegionEntryHelper needs unit test Added tests for OffHeapRegionEntryHelper. You can merge this pull request into a Git repository by running: $ git pull https://g

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

Github user sboorlagadda commented on the pull request: https://github.com/apache/incubator-geode/pull/83#issuecomment-176991959 Review board link -> https://reviews.apache.org/r/42884/ --- If your project is set up for it, you can reply to this email and have your reply appear on Git

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42977/#review117035 --- Ship it! Ship It! - Dan Smith On Jan. 29, 2016, 9:34 p.m., B

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42977/#review117038 --- gemfire-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan

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

+1 --Mark On Thu, Jan 28, 2016 at 6:40 PM, John Blum wrote: > +1 > > I would also add, in addition to what Dan mentioned, that GEODE-27 get > fixed sooner rather than later. I recently completed the Spring Session > GemFire support and went ahead to add support for Apache Geode as well. I > c

gemfire-junit

On a different thread, Niall pointed out: > On Jan 19, 2016, at 8:07 PM, Niall Pemberton > wrote: > > * The gemfire-junit-1.0.0-incubating.M1.jar is empty so would be better to > not create this maven artifact Looking at the gemfire-junit project I think we should move the code into src/main/

Re: gemfire-junit

+1 -Dan On Fri, Jan 29, 2016 at 3:05 PM, Anthony Baker wrote: > On a different thread, Niall pointed out: > > > On Jan 19, 2016, at 8:07 PM, Niall Pemberton > wrote: > > > > * The gemfire-junit-1.0.0-incubating.M1.jar is empty so would be better > to > > not create this maven artifact > > Look

Re: gemfire-junit

+1 On Fri, Jan 29, 2016 at 3:26 PM, Dan Smith wrote: > +1 > > -Dan > > On Fri, Jan 29, 2016 at 3:05 PM, Anthony Baker wrote: > > > On a different thread, Niall pointed out: > > > > > On Jan 19, 2016, at 8:07 PM, Niall Pemberton < > niall.pember...@gmail.com> > > wrote: > > > > > > * The gemfir

Review Request 42991: GEODE-805: Fix javadoc warnings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42991/ --- Review request for geode. Bugs: GEODE-805 https://issues.apache.org/jira/br

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

+1 On 30/01/2016 9:57 am, Mark Bretl wrote: +1 --Mark On Thu, Jan 28, 2016 at 6:40 PM, John Blum wrote: +1 I would also add, in addition to what Dan mentioned, that GEODE-27 get fixed sooner rather than later. I recently completed the Spring Session GemFire support and went ahead to add s

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

Please review this (I didn't specify any specific reviewers). I'd like to get this merged to develop sooner than later since it touches so many files and I'd like to avoid merge conflicts if possible. You can also checkout feature/GEODE-805 if you want to build or edit the changes. https://reviews

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42991/ --- (Updated Jan. 30, 2016, 12:44 a.m.) Review request for geode. Changes ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42991/ --- (Updated Jan. 30, 2016, 12:58 a.m.) Review request for geode. Changes ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42991/#review117073 --- Apparently the warning about javax.transaction-api can be elminate

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

--- Spring Data GemFire > Nightly-ApacheGeode > #205 was successful. --- Scheduled 1226 tests in total. https://build.spring.io/browse/SGF-NAG-205/ -- This