Build failed in Jenkins: Geode-nightly #657

See Changes: [klund] GEODE-2126: apply FlakyTest category to flaky test [gzhou] GEODE-2120: The test needs to wait longer for compaction to finish. -- [...truncated 513 lines...] Note: Recompile wi

build failure - MultipleOplogsRollingFeatureJUnitTest

All of the builds are failing in spotless checks on this class.

Re: build failure - MultipleOplogsRollingFeatureJUnitTest

fixed On Fri, Nov 18, 2016 at 8:44 AM, Bruce Schuchardt wrote: > All of the builds are failing in spotless checks on this class. >

Re: Review Request 53867: GEODE-2129 make pdxtype id random(not sequential)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53867/#review156309 --- geode-core/src/main/java/org/apache/geode/pdx/internal/PeerTypeRe

[GitHub] incubator-geode pull request #287: GEODE-2112: Fix UITests failing due to in...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/287 --- 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 featur

[GitHub] incubator-geode pull request #289: GEODE-2117: Pulse handles float type mbea...

GitHub user jaredjstewart opened a pull request: https://github.com/apache/incubator-geode/pull/289 GEODE-2117: Pulse handles float type mbean attributes - Added tests for JMXDataUpdater::getDoubleAttribute() - JMXDataUpdater::getDoubleAttribute() now returns double approximati

[GitHub] incubator-geode issue #289: GEODE-2117: Pulse handles float type mbean attri...

Github user jaredjstewart commented on the issue: https://github.com/apache/incubator-geode/pull/289 Precheckin started --- 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 an

Re: Review Request 53865: GEODE-2127 old client support service is not set up to handle arrays

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53865/#review156322 --- Ship it! Ship It! - Hitesh Khamesra On Nov. 17, 2016, 11:33

Re: Review Request 53864: GEODE-2121 add DLockTest, MembershipTest, ClientServerTest, ClientSubscriptionTest and RestAPITest categories

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53864/#review156329 --- Ship it! Ship It! - Hitesh Khamesra On Nov. 17, 2016, 11:07

PartitionedRegionLoadModelJUnitTest > testRedundancySatisfactionPreferRemoteIp FAILED

I'm receiving the following assertion error when compiling: org.apache.geode.internal.cache.partitioned.PartitionedRegionLoadModelJUnitTest > testRedundancySatisfactionPreferRemoteIp FAILED java.lang.AssertionError: expected:<[Create[member=10. 64.251.218(1639):3,bucketId=0], Create[member=10

Re: PartitionedRegionLoadModelJUnitTest > testRedundancySatisfactionPreferRemoteIp FAILED

On your home network is kduling-mbpro a valid hostname? Anthony > On Nov 18, 2016, at 1:00 PM, Kevin Duling wrote: > > I'm receiving the following assertion error when compiling: > > org.apache.geode.internal.cache.partitioned.PartitionedRegionLoadModelJUnitTest >> testRedundancySatisfactionPr

[GitHub] incubator-geode pull request #288: GEODE-2092 - Security examples should not...

Github user kjduling closed the pull request at: https://github.com/apache/incubator-geode/pull/288 --- 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

[GitHub] incubator-geode issue #288: GEODE-2092 - Security examples should not be in ...

Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/288 Cancelling this one to resubmit as a single, clean commit. --- 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 do

[GitHub] incubator-geode pull request #290: GEODE-2092: Security examples should not ...

GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/290 GEODE-2092: Security examples should not be in the product code Moved examples to geode-core/src/main/java/org/apache/geode/examples/security/ Copied old Sample/Simple code to test pac

Someone to pull in #288

Can someone else please pull in #288, resolve conflicts and rebase it into one commit? When I attempt to do this, the "author" gets reset to me instead of Kevin. My git knowledge isn't good enough for multi-commit PRs that include conflicts with develop. -Kirk

Re: Someone to pull in #288

Needs to be #290. I closed 288 in an attempt to have it merge cleaner. What should happen: SimpleSecurityManager -> moved to ./geode-core/src/test/java/org/apache/geode/security/SimpleTestSecurityManager.java SampleSecurityManager -> moved to ./geode-core/src/test/java/org/apache/geode/security/

Cluster Configuration files

The cluster.xml and cluster.properties files seem to contain the entirety of a given cluster configuration. These are the only files included in the zip file produced by ‘export cluster-configuration’. Does anyone know what the ConfigDiskDir files are used for? Are they just redundant version

[GitHub] incubator-geode issue #289: GEODE-2117: Pulse handles float type mbean attri...

Github user jaredjstewart commented on the issue: https://github.com/apache/incubator-geode/pull/289 Precheckin passed except for a FlakyTest: ``` org.apache.geode.management.internal.cli.commands.IndexCommandsDUnitTest > testCreateDestroyUpdatesSharedConfig FAILED

Re: Review Request 53867: GEODE-2129 make pdxtype id random(not sequential)

Before using hashcode for pdxType creation, we will check whether it is available or not. If it's not available then, we will use next available id. We have introduced this mechanism to harden the cases where the user ends up having two clusters with the same dsid.  From: Vishal Rao To:

Re: Cluster Configuration files

As far as I remember, cluster configurations files are saved in an internal region as well, and that region is defined as replicate_persistent, so the content of it gets written to the disk as well in those ConfigDistDir backup files. This is something we need to clean up as well. Just use one way

Review Request 53910: GEODE-2124: Queries that invoke StructSet addAll fail with ClassCastException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53910/ --- Review request for geode, Barry Oglesby and Dan Smith. Repository: geode Desc

Re: Cluster Configuration files

In my memory the reason the files are written to disk is because nobody wrote a input/output stream or other io interfaces for the XML parser to use. Easy enough to do through. On Fri, Nov 18, 2016 at 3:50 PM Jinmei Liao wrote: > As far as I remember, cluster configurations files are saved in an

Review Request 53917: GEODE-2123 Replace static members in ParallelgatewaySenderQueues with instance variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53917/ --- Review request for geode, Barry Oglesby, Jason Huynh, Dan Smith, and xiaojian zh

Re: Review Request 53910: GEODE-2124: Queries that invoke StructSet addAll fail with ClassCastException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53910/#review156359 --- Ship it! Ship It! - Barry Oglesby On Nov. 19, 2016, 12:01 a.

Re: Review Request 53910: GEODE-2124: Queries that invoke StructSet addAll fail with ClassCastException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53910/#review156360 --- Fix it, then Ship it! geode-core/src/main/java/org/apache/geod

[GitHub] incubator-geode pull request #289: GEODE-2117: Pulse handles float type mbea...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/289 --- 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 featur

Re: Review Request 53917: GEODE-2123 Replace static members in ParallelgatewaySenderQueues with instance variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53917/#review156363 --- geode-core/src/main/java/org/apache/geode/internal/cache/wan/para

Re: Review Request 53917: GEODE-2123 Replace static members in ParallelgatewaySenderQueues with instance variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53917/#review156370 --- geode-core/src/main/java/org/apache/geode/internal/cache/wan/para

Re: Review Request 53917: GEODE-2123 Replace static members in ParallelgatewaySenderQueues with instance variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53917/ --- (Updated Nov. 19, 2016, 1:12 a.m.) Review request for geode, Barry Oglesby, Jas

Re: Review Request 53917: GEODE-2123 Replace static members in ParallelgatewaySenderQueues with instance variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53917/ --- (Updated Nov. 19, 2016, 1:14 a.m.) Review request for geode, Barry Oglesby, Jas