Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 898C2200B8C for ; Mon, 12 Sep 2016 21:30:49 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 88474160AB2; Mon, 12 Sep 2016 19:30:49 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 81062160AB8 for ; Mon, 12 Sep 2016 21:30:48 +0200 (CEST) Received: (qmail 9659 invoked by uid 500); 12 Sep 2016 19:30:47 -0000 Mailing-List: contact commits-help@geode.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.incubator.apache.org Delivered-To: mailing list commits@geode.incubator.apache.org Received: (qmail 9650 invoked by uid 99); 12 Sep 2016 19:30:47 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Sep 2016 19:30:47 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 325B018014C for ; Mon, 12 Sep 2016 19:30:47 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -4.646 X-Spam-Level: X-Spam-Status: No, score=-4.646 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, KAM_LAZY_DOMAIN_SECURITY=1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id oABSo6C1PWr7 for ; Mon, 12 Sep 2016 19:30:45 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with SMTP id BFA4B60DC6 for ; Mon, 12 Sep 2016 19:30:43 +0000 (UTC) Received: (qmail 8942 invoked by uid 99); 12 Sep 2016 19:30:43 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Sep 2016 19:30:43 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id B5F1BE78A1; Mon, 12 Sep 2016 19:30:42 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: udo@apache.org To: commits@geode.incubator.apache.org Date: Mon, 12 Sep 2016 19:30:50 -0000 Message-Id: <8c726a2c56a44d6da3a392d106d33bd1@git.apache.org> In-Reply-To: <93a8db508ded4bba937aa97259a5042a@git.apache.org> References: <93a8db508ded4bba937aa97259a5042a@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [09/11] incubator-geode git commit: Fixed eclipse compilation and test execution issues archived-at: Mon, 12 Sep 2016 19:30:49 -0000 Fixed eclipse compilation and test execution issues With the eclipse compiler, AsyncEventListenerDUnitTest was generating a compilation error about a deserializeLambda method being to large. This doesn't seem to be a real issue because the javac doesn't have the same problem. However, by refactoring the test slightly I was able to make this error go away. Changed the eclipse classpath to include the generated-resources directories. These directories contain files like GemFireVersion.properties that need to be included on the classpath, but we were filtering them out. Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/e4a09a6a Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/e4a09a6a Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/e4a09a6a Branch: refs/heads/feature/GEODE-420 Commit: e4a09a6a784ff729c006dba28ec697fd89ac86cd Parents: 182cacb Author: Dan Smith Authored: Tue Aug 30 16:36:59 2016 -0700 Committer: Dan Smith Committed: Fri Sep 9 13:27:47 2016 -0700 ---------------------------------------------------------------------- .../asyncqueue/AsyncEventListenerDUnitTest.java | 74 +++++++------------- gradle/ide.gradle | 11 +-- 2 files changed, 33 insertions(+), 52 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/e4a09a6a/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java index 4c78bdc..35f6810 100644 --- a/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java +++ b/geode-core/src/test/java/com/gemstone/gemfire/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java @@ -732,10 +732,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm3.invoke(createAsyncEventQueueRunnable()); vm4.invoke(createAsyncEventQueueRunnable()); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1.invoke(() -> AsyncEventQueueTestBase.doPuts( getTestMethodName() + "_PR", 500 )); @@ -771,10 +768,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", false, 100, 100, true, false, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1 .invoke(pauseAsyncEventQueueRunnable()); @@ -856,10 +850,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", false, 100, 100, false, true, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1.invoke(() -> AsyncEventQueueTestBase.doPuts( getTestMethodName() + "_PR", 500 )); @@ -960,18 +951,12 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", true, 100, 100, false, false, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1.invoke(() -> AsyncEventQueueTestBase.doPuts( getTestMethodName() + "_PR", 256 )); - vm1.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm2.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm3.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm4.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + waitForAsyncQueueToGetEmpty(); int vm1size = (Integer)vm1.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); int vm2size = (Integer)vm2.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); @@ -981,6 +966,20 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { assertEquals(vm1size + vm2size + vm3size + vm4size, 256); } +private void createPartitionedRegionWithAsyncEventQueue() { + vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); +} + +private void waitForAsyncQueueToGetEmpty() { + vm1.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + vm2.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + vm3.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + vm4.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); +} + @Test public void testParallelAsyncEventQueueWithSubstitutionFilter() { Integer lnPort = (Integer)vm0.invoke(() -> AsyncEventQueueTestBase.createFirstLocatorWithDSId( 1 )); @@ -1074,10 +1073,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", true, 100, 100, false, false, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1 .invoke(pauseAsyncEventQueueRunnable()); @@ -1121,10 +1117,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createConcurrentAsyncEventQueue( "ln", true, 100, 100, false, false, null, false, 2, OrderPolicy.KEY )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1 .invoke(pauseAsyncEventQueueRunnable()); @@ -1164,10 +1157,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", true, 100, 100, true, false, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1 .invoke(pauseAsyncEventQueueRunnable()); @@ -1215,10 +1205,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm3.invoke(() -> AsyncEventQueueTestBase.resumeAsyncEventQueue( "ln" )); vm4.invoke(() -> AsyncEventQueueTestBase.resumeAsyncEventQueue( "ln" )); - vm1.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm2.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm3.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm4.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + waitForAsyncQueueToGetEmpty(); int vm1size = (Integer)vm1.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); int vm2size = (Integer)vm2.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); @@ -1295,10 +1282,7 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm3.invoke(() -> AsyncEventQueueTestBase.resumeAsyncEventQueue( "ln" )); vm4.invoke(() -> AsyncEventQueueTestBase.resumeAsyncEventQueue( "ln" )); - vm1.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm2.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm3.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm4.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + waitForAsyncQueueToGetEmpty(); int vm1size = (Integer)vm1.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); int vm2size = (Integer)vm2.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); @@ -1368,18 +1352,12 @@ public class AsyncEventListenerDUnitTest extends AsyncEventQueueTestBase { vm4.invoke(() -> AsyncEventQueueTestBase.createAsyncEventQueue( "ln", true, 100, 100, false, true, null, false )); - vm1.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm2.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm3.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); - vm4.invoke(() -> AsyncEventQueueTestBase.createPartitionedRegionWithAsyncEventQueue( getTestMethodName() + "_PR", "ln", isOffHeap() )); + createPartitionedRegionWithAsyncEventQueue(); vm1.invoke(() -> AsyncEventQueueTestBase.doPuts( getTestMethodName() + "_PR", 256 )); - vm1.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm2.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm3.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); - vm4.invoke(() -> AsyncEventQueueTestBase.waitForAsyncQueueToGetEmpty( "ln" )); + waitForAsyncQueueToGetEmpty(); int vm1size = (Integer)vm1.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); int vm2size = (Integer)vm2.invoke(() -> AsyncEventQueueTestBase.getAsyncEventListenerMapSize( "ln")); http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/e4a09a6a/gradle/ide.gradle ---------------------------------------------------------------------- diff --git a/gradle/ide.gradle b/gradle/ide.gradle index d07ad0b..a42b92b 100644 --- a/gradle/ide.gradle +++ b/gradle/ide.gradle @@ -31,10 +31,13 @@ subprojects { // Remove the gradle output directories from the eclipse classpath. // Unfortunately, using minusConfigurations does not work here, because // it removes the entire geode-core project. - classpath.entries.removeAll { entry -> entry.path.contains('geode-core/build')} - classpath.entries.removeAll { entry -> entry.path.contains('geode-web/build')} - classpath.entries.removeAll { entry -> entry.path.contains('geode-assembly/build')} - classpath.entries.removeAll { entry -> entry.path.contains('geode-dependencies.jar')} + classpath.entries.removeAll { entry -> + (entry.path.contains('geode-core/build') || + entry.path.contains('geode-web/build') || + entry.path.contains('geode-assembly/build') || + entry.path.contains('geode-dependencies.jar')) && + !entry.path.contains('generated-resources') + } //By default, gradle adds the java 1.8 *execution environment*, which has access restrictions on //things like Unsafe. Change it to a direct dependency on the workspace JDK