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

2016-08-03 Thread Spring CI 

---
Spring Data GemFire > Nightly-ApacheGeode > #390 was successful.
---
Scheduled
1425 tests in total.

https://build.spring.io/browse/SGF-NAG-390/





--
This message is automatically generated by Atlassian Bamboo

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Barry Oglesby 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50680/#review144660
---


Ship it!




Ship It!

- Barry Oglesby


On Aug. 3, 2016, 6:36 p.m., Darrel Schneider wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50680/
> ---
> 
> (Updated Aug. 3, 2016, 6:36 p.m.)
> 
> 
> Review request for geode, Barry Oglesby, Eric Shu, Scott Jewell, and Ken Howe.
> 
> 
> Bugs: GEODE-1718
> https://issues.apache.org/jira/browse/GEODE-1718
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> New unit test demonstrated that replace would return false when done on an 
> overflowed entry.
> With the fix the test passes.
> Also added a null check in checkPdxEquals to prevent NPE if obj is ever null.
> If obj is null checkPdxEquals will now call basicEquals which handles testing 
> null.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionEntry.java
>  15a5bed750b676c668ddde64291a5b315fca90fd 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionMap.java
>  f3cb3d654087c797de430f448348c3ba66846ac4 
>   
> geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ReplaceWithOverflowJUnitTest.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/50680/diff/
> 
> 
> Testing
> ---
> 
> precheckin
> 
> 
> Thanks,
> 
> Darrel Schneider
> 
>

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider 


> On Aug. 3, 2016, 11:35 a.m., Darrel Schneider wrote:
> > This fix has a problem. It always reads the value from disk if it had 
> > overflowed.
> > But just a couple of lines before this basicPut has called 
> > setOldValueInEvent.
> > I think the correct fix is to use that old value from the event instead of 
> > rereading from disk/memory.

I have posted a new review that has a fix that does not result in reading the 
value twice


- Darrel


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50680/#review144640
---


On Aug. 3, 2016, 11:36 a.m., Darrel Schneider wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50680/
> ---
> 
> (Updated Aug. 3, 2016, 11:36 a.m.)
> 
> 
> Review request for geode, Barry Oglesby, Eric Shu, Scott Jewell, and Ken Howe.
> 
> 
> Bugs: GEODE-1718
> https://issues.apache.org/jira/browse/GEODE-1718
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> New unit test demonstrated that replace would return false when done on an 
> overflowed entry.
> With the fix the test passes.
> Also added a null check in checkPdxEquals to prevent NPE if obj is ever null.
> If obj is null checkPdxEquals will now call basicEquals which handles testing 
> null.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionEntry.java
>  15a5bed750b676c668ddde64291a5b315fca90fd 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionMap.java
>  f3cb3d654087c797de430f448348c3ba66846ac4 
>   
> geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ReplaceWithOverflowJUnitTest.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/50680/diff/
> 
> 
> Testing
> ---
> 
> precheckin
> 
> 
> Thanks,
> 
> Darrel Schneider
> 
>

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50680/
---

(Updated Aug. 3, 2016, 11:36 a.m.)


Review request for geode, Barry Oglesby, Eric Shu, Scott Jewell, and Ken Howe.


Bugs: GEODE-1718
https://issues.apache.org/jira/browse/GEODE-1718


Repository: geode


Description
---

New unit test demonstrated that replace would return false when done on an 
overflowed entry.
With the fix the test passes.
Also added a null check in checkPdxEquals to prevent NPE if obj is ever null.
If obj is null checkPdxEquals will now call basicEquals which handles testing 
null.


Diffs (updated)
-

  
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionEntry.java
 15a5bed750b676c668ddde64291a5b315fca90fd 
  
geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionMap.java
 f3cb3d654087c797de430f448348c3ba66846ac4 
  
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ReplaceWithOverflowJUnitTest.java
 PRE-CREATION 

Diff: https://reviews.apache.org/r/50680/diff/


Testing
---

precheckin


Thanks,

Darrel Schneider

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50680/#review144640
---



This fix has a problem. It always reads the value from disk if it had 
overflowed.
But just a couple of lines before this basicPut has called setOldValueInEvent.
I think the correct fix is to use that old value from the event instead of 
rereading from disk/memory.

- Darrel Schneider


On Aug. 1, 2016, 4:17 p.m., Darrel Schneider wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50680/
> ---
> 
> (Updated Aug. 1, 2016, 4:17 p.m.)
> 
> 
> Review request for geode, Barry Oglesby, Eric Shu, Scott Jewell, and Ken Howe.
> 
> 
> Bugs: GEODE-1718
> https://issues.apache.org/jira/browse/GEODE-1718
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> New unit test demonstrated that replace would return false when done on an 
> overflowed entry.
> With the fix the test passes.
> Also added a null check in checkPdxEquals to prevent NPE if obj is ever null.
> If obj is null checkPdxEquals will now call basicEquals which handles testing 
> null.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionEntry.java
>  15a5bed750b676c668ddde64291a5b315fca90fd 
>   
> geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionMap.java
>  f3cb3d654087c797de430f448348c3ba66846ac4 
>   
> geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ReplaceWithOverflowJUnitTest.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/50680/diff/
> 
> 
> Testing
> ---
> 
> precheckin
> 
> 
> Thanks,
> 
> Darrel Schneider
> 
>

[GitHub] incubator-geode pull request #226: GEODE-1712: introduce SecurityService int...

2016-08-03 Thread gracemeilen 
GitHub user gracemeilen opened a pull request:

https://github.com/apache/incubator-geode/pull/226

GEODE-1712: introduce SecurityService interface for mocking

GEODE-1712: add missing categories and fix test failure
* fix changes recommended in review
GEODE-1712: add static factory methods to SecurityService
GEODE-1713: cleanup imports
GEODE-1712: introduce SecurityService interface for mocking
* create Unit Tests for client commands that use security

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gracemeilen/incubator-geode 
feature/GEODE-1712-3

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-geode/pull/226.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #226


commit 449e5c4c3dcc26b982fa822b9a8d210c6dd9732d
Author: gmeilen 
Date:   2016-07-18T20:13:11Z

GEODE-1712: introduce SecurityService interface for mocking

* create Unit Tests for client commands that use security

commit b136c20417bdff2b9e7c6778471b12f48e2aedd2
Author: gmeilen 
Date:   2016-07-28T17:46:30Z

GEODE-1713: cleanup imports

commit b549bcc14ee2eeb1c0d5895965b3082875219fa5
Author: gmeilen 
Date:   2016-07-28T20:20:23Z

GEODE-1712: add static factory methods to SecurityService

commit cb08cfcee874fe67e5a64518e08b7e517d59f04c
Author: gmeilen 
Date:   2016-08-03T18:15:55Z

GEODE-1712: add missing categories and fix test failures




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request #223: GEODE-1712: introduce SecurityService and...

2016-08-03 Thread gracemeilen 
Github user gracemeilen closed the pull request at:

https://github.com/apache/incubator-geode/pull/223


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Build failed in Jenkins: Geode-release #23

2016-08-03 Thread Apache Jenkins Server 
See 

--
[...truncated 801 lines...]
:geode-lucene:compileJavaNote: Some input files use or override a deprecated 
API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-lucene:processResources
:geode-lucene:classes
:geode-pulse:compileJavaNote: Some input files use unchecked or unsafe 
operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-pulse:copyGemFireVersionFile
:geode-pulse:processResources
:geode-pulse:classes
:geode-rebalancer:compileJava
:geode-rebalancer:processResources UP-TO-DATE
:geode-rebalancer:classes
:geode-wan:compileJava
:geode-wan:processResources
:geode-wan:classes
:geode-web:compileJava UP-TO-DATE
:geode-web:processResources UP-TO-DATE
:geode-web:classes UP-TO-DATE
:geode-web-api:compileJavaNote: Some input files use unchecked or unsafe 
operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-web-api:processResources UP-TO-DATE
:geode-web-api:classes
:geode-assembly:docs
:geode-assembly:gfshDepsJar
:geode-common:javadocJar
:geode-common:sourcesJar
:geode-common:signArchives SKIPPED
:geode-core:javadocJar
:geode-core:raJar
:geode-core:jcaJar
:geode-core:sourcesJar
:geode-core:signArchives SKIPPED
:geode-core:webJar
:geode-cq:jar
:geode-cq:javadoc
:geode-cq:javadocJar
:geode-cq:sourcesJar
:geode-cq:signArchives SKIPPED
:geode-json:javadocJar
:geode-json:sourcesJar
:geode-json:signArchives SKIPPED
:geode-lucene:jar
:geode-lucene:javadoc
:geode-lucene:javadocJar
:geode-lucene:sourcesJar
:geode-lucene:signArchives SKIPPED
:geode-pulse:javadoc
:geode-pulse:javadocJar
:geode-pulse:sourcesJar
:geode-pulse:war
:geode-pulse:signArchives SKIPPED
:geode-wan:jar
:geode-wan:javadoc
:geode-wan:javadocJar
:geode-wan:sourcesJar
:geode-wan:signArchives SKIPPED
:geode-web:javadoc UP-TO-DATE
:geode-web:javadocJar
:geode-web:sourcesJar
:geode-web:war
:geode-web:signArchives SKIPPED
:geode-web-api:jar
:geode-web-api:javadoc
:geode-web-api:javadocJar
:geode-web-api:sourcesJar
:geode-web-api:war
:geode-web-api:signArchives SKIPPED
:geode-assembly:distTar
:geode-assembly:distZip
:geode-assembly:writeBuildInfo
:geode-assembly:srcDistTar
:geode-assembly:srcDistZip
:geode-assembly:signArchives SKIPPED
:geode-assembly:assemble
:geode-assembly:compileTestJava
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpclient/4.5.1/httpclient-4.5.1.pom
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpcomponents-client/4.5.1/httpcomponents-client-4.5.1.pom
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpcore/4.4.3/httpcore-4.4.3.pom
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpcomponents-core/4.4.3/httpcomponents-core-4.4.3.pom
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpclient/4.5.1/httpclient-4.5.1.jar
Download 
https://repo1.maven.org/maven2/org/apache/httpcomponents/httpcore/4.4.3/httpcore-4.4.3.jar
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:geode-assembly:processTestResources
:geode-assembly:testClasses
:geode-assembly:checkMissedTests
:geode-assembly:installDist
:geode-assembly:test
:geode-assembly:check
:geode-assembly:build
:geode-assembly:distributedTest
:geode-assembly:flakyTest
:geode-assembly:integrationTest
:geode-common:assemble
:geode-common:compileTestJava
:geode-common:processTestResources UP-TO-DATE
:geode-common:testClasses
:geode-common:checkMissedTests
:geode-common:test
:geode-common:check
:geode-common:build
:geode-common:distributedTest
:geode-common:flakyTest
:geode-common:integrationTest
:geode-core:assemble
:geode-core:checkMissedTests
Download 
https://repo1.maven.org/maven2/org/apache/derby/derby/10.2.2.0/derby-10.2.2.0.pom
Download 
https://repo1.maven.org/maven2/org/apache/derby/derby/10.2.2.0/derby-10.2.2.0.jar
:geode-core:test
:geode-core:check
:geode-core:build
:geode-core:distributedTest
:geode-core:flakyTest
:geode-core:integrationTest

org.apache.geode.redis.HashesJUnitTest > testHMSetHSetHLen FAILED
redis.clients.jedis.exceptions.JedisConnectionException: 
java.net.SocketTimeoutException: Read timed out
at 
redis.clients.util.RedisInputStream.ensureFill(RedisInputStream.java:201)
at 
redis.clients.util.RedisInputStream.readByte(RedisInputStream.java:40)
at redis.clients.jedis.Protocol.process(Protocol.java:132)
at redis.clients.jedis.Protocol.read(Protocol.java:196)
at 
redis.clients.jedis.Connection.readProtocolWithCheckingBroken(Connection.java:288)
at redis.clients.jedis.Connection.getIntegerReply(Connection.java:213)
at redis.clients.jedis.Jedis.hset(Jedis.java:590)
at

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

2016-08-03 Thread Bruce Schuchardt 
The port-range test failure is due to my checkin yesterday.  I'm looking 
into it.


Le 8/3/2016 à 7:28 AM, Apache Jenkins Server a écrit :

See 

Changes:

[jdeppe] GEODE-1666: Bump Gradle from 2.12 to 2.14.1

[klund] GEODE-1682: Adding options for starting Geode REST API using gfsh

[klund] GEODE-1682: Added test for the REST arguments and Use correct

[klund] GEODE-1682: Fixing test failure HelpCommandsIntegrationTest >

[bschuchardt] restricting membership-port-range for tests to be outside of

[bschuchardt] GC thread names have changed.  This fixes the expected names in

[bschuchardt] GEODE-1645 forceUDPMessagingForCurrentThread should be re-enabled

--
[...truncated 566 lines...]
:geode-common:build
:geode-common:distributedTest
:geode-common:flakyTest
:geode-common:integrationTest
:geode-core:assemble
:geode-core:checkMissedTests
:geode-core:test
:geode-core:check
:geode-core:build
:geode-core:distributedTest

com.gemstone.gemfire.distributed.DistributedSystemDUnitTest > 
testConflictingUDPPort FAILED
 java.lang.IllegalArgumentException: Could not set "membership-port-range" to 
"21,017" because its value can not be less than "32,769".
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.minMaxCheck(AbstractDistributionConfig.java:100)
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkMembershipPortRange(AbstractDistributionConfig.java:383)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkAttribute(AbstractDistributionConfig.java:84)
 at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.setMembershipPortRange(DistributionConfigImpl.java:3172)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.setAttributeObject(AbstractDistributionConfig.java:564)
 at 
com.gemstone.gemfire.internal.AbstractConfig.setAttribute(AbstractConfig.java:394)
 at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.initialize(DistributionConfigImpl.java:1322)
 at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.(DistributionConfigImpl.java:734)
 at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.(DistributionConfigImpl.java:623)
 at 
com.gemstone.gemfire.distributed.internal.InternalDistributedSystem.(InternalDistributedSystem.java:409)
 at 
com.gemstone.gemfire.distributed.internal.InternalDistributedSystem.newInstance(InternalDistributedSystem.java:240)
 at 
com.gemstone.gemfire.distributed.DistributedSystem.connect(DistributedSystem.java:238)
 at 
com.gemstone.gemfire.distributed.DistributedSystemDUnitTest.testConflictingUDPPort(DistributedSystemDUnitTest.java:368)

com.gemstone.gemfire.distributed.DistributedSystemDUnitTest > testUDPPortRange 
FAILED
 java.lang.IllegalArgumentException: Could not set "membership-port-range" to 
"20,046" because its value can not be less than "32,769".
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.minMaxCheck(AbstractDistributionConfig.java:100)
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkMembershipPortRange(AbstractDistributionConfig.java:383)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkAttribute(AbstractDistributionConfig.java:84)
 at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.setMembershipPortRange(DistributionConfigImpl.java:3172)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:498)
 at

Build failed in Jenkins: Geode-nightly #549

2016-08-03 Thread Apache Jenkins Server 
See 

Changes:

[jdeppe] GEODE-1666: Bump Gradle from 2.12 to 2.14.1

[klund] GEODE-1682: Adding options for starting Geode REST API using gfsh

[klund] GEODE-1682: Added test for the REST arguments and Use correct

[klund] GEODE-1682: Fixing test failure HelpCommandsIntegrationTest >

[bschuchardt] restricting membership-port-range for tests to be outside of

[bschuchardt] GC thread names have changed.  This fixes the expected names in

[bschuchardt] GEODE-1645 forceUDPMessagingForCurrentThread should be re-enabled

--
[...truncated 566 lines...]
:geode-common:build
:geode-common:distributedTest
:geode-common:flakyTest
:geode-common:integrationTest
:geode-core:assemble
:geode-core:checkMissedTests
:geode-core:test
:geode-core:check
:geode-core:build
:geode-core:distributedTest

com.gemstone.gemfire.distributed.DistributedSystemDUnitTest > 
testConflictingUDPPort FAILED
java.lang.IllegalArgumentException: Could not set "membership-port-range" 
to "21,017" because its value can not be less than "32,769".
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.minMaxCheck(AbstractDistributionConfig.java:100)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkMembershipPortRange(AbstractDistributionConfig.java:383)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkAttribute(AbstractDistributionConfig.java:84)
at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.setMembershipPortRange(DistributionConfigImpl.java:3172)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.setAttributeObject(AbstractDistributionConfig.java:564)
at 
com.gemstone.gemfire.internal.AbstractConfig.setAttribute(AbstractConfig.java:394)
at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.initialize(DistributionConfigImpl.java:1322)
at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.(DistributionConfigImpl.java:734)
at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.(DistributionConfigImpl.java:623)
at 
com.gemstone.gemfire.distributed.internal.InternalDistributedSystem.(InternalDistributedSystem.java:409)
at 
com.gemstone.gemfire.distributed.internal.InternalDistributedSystem.newInstance(InternalDistributedSystem.java:240)
at 
com.gemstone.gemfire.distributed.DistributedSystem.connect(DistributedSystem.java:238)
at 
com.gemstone.gemfire.distributed.DistributedSystemDUnitTest.testConflictingUDPPort(DistributedSystemDUnitTest.java:368)

com.gemstone.gemfire.distributed.DistributedSystemDUnitTest > testUDPPortRange 
FAILED
java.lang.IllegalArgumentException: Could not set "membership-port-range" 
to "20,046" because its value can not be less than "32,769".
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.minMaxCheck(AbstractDistributionConfig.java:100)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkMembershipPortRange(AbstractDistributionConfig.java:383)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.checkAttribute(AbstractDistributionConfig.java:84)
at 
com.gemstone.gemfire.distributed.internal.DistributionConfigImpl.setMembershipPortRange(DistributionConfigImpl.java:3172)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
com.gemstone.gemfire.distributed.internal.AbstractDistributionConfig.setAttributeObject(AbstractDistributionConfig.java:564)
at 
com.gemstone.gemfire.internal.AbstractConfig.setAttribute(AbstractConfig.java:394)
at

[GitHub] incubator-geode pull request #225: Fix e.printStackTrace() in ClientPartitio...

2016-08-03 Thread omallassi 
GitHub user omallassi opened a pull request:

https://github.com/apache/incubator-geode/pull/225

Fix e.printStackTrace() in ClientPartitionAdvisor

the exception was logged using e.printstacktrace(), switch to use the
LoggerService.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/omallassi/incubator-geode develop

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-geode/pull/225.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #225


commit 9d3bbad1a51df290ccd686ff6f0f544dd9c2617f
Author: Olivier Mallassi 
Date:   2016-08-03T08:28:38Z

Fix e.printStackTrace() in ClientPartitionAdvisor

the exception was logged using e.printstacktrace(), switch to use the
LoggerService.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---