[geode] branch develop updated (dcca237 -> c5a3b16)

2020-02-12 Thread bschuchardt
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.


from dcca237  GEODE-7760: fixing 
LocatorUDPSecurityDUnitTest.testCrashLocatorMultip… (#4683)
 add c5a3b16  GEODE-7792: configure logging for geode-membership 
integration tests (#4688)

No new revisions were added by this update.

Summary of changes:
 .../src/integrationTest/resources/log4j2.xml | 12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)
 copy geode-log4j/src/main/resources/log4j2-cli.xml => 
geode-membership/src/integrationTest/resources/log4j2.xml (80%)
 mode change 100644 => 100755



[geode] branch develop updated (5e8107a -> dcca237)

2020-02-12 Thread bschuchardt
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 5e8107a  GEODE-7788: Alpine tools docker image creation fails due to 
internal (#4685)
 add dcca237  GEODE-7760: fixing 
LocatorUDPSecurityDUnitTest.testCrashLocatorMultip… (#4683)

No new revisions were added by this update.

Summary of changes:
 .../distributed/LocatorUDPSecurityDUnitTest.java |  1 +
 .../gms/messenger/GMSQuorumCheckerJUnitTest.java | 20 ++--
 .../gms/messenger/JGroupsMessengerJUnitTest.java |  2 +-
 .../membership/gms/messenger/GMSEncrypt.java |  7 ---
 .../membership/gms/messenger/GMSQuorumChecker.java   |  8 ++--
 .../membership/gms/messenger/JGroupsMessenger.java   |  6 --
 .../gms/messenger/MembershipInformationImpl.java | 10 +-
 7 files changed, 35 insertions(+), 19 deletions(-)



[geode] branch release/1.12.0 updated: GEODE-7788: Alpine tools docker image creation fails due to internal (#4685)

2020-02-12 Thread echobravo
This is an automated email from the ASF dual-hosted git repository.

echobravo pushed a commit to branch release/1.12.0
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/release/1.12.0 by this push:
 new 1865181  GEODE-7788: Alpine tools docker image creation fails due to 
internal (#4685)
1865181 is described below

commit 1865181b8e7395f05899aefa9d914122e795d052
Author: Bala Kaza Venkata <43208983+balak...@users.noreply.github.com>
AuthorDate: Tue Feb 11 16:54:55 2020 -0500

GEODE-7788: Alpine tools docker image creation fails due to internal (#4685)

winrm-cli changes.

Change the dockerfile to do go get instead of git clone which does
cloning, pulling dependencies and building as part of its process.
This eliminates the manual winrm-cli build process that was
being done before.

Signed-off-by: Bala Kaza Venkata 

Co-authored-by: Robert Houghton 
(cherry picked from commit 5e8107ad12e4ba891bb7707f111696282b2cff2a)
---
 ci/images/alpine-tools/Dockerfile | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/ci/images/alpine-tools/Dockerfile 
b/ci/images/alpine-tools/Dockerfile
index 80ccb87..49ee247 100644
--- a/ci/images/alpine-tools/Dockerfile
+++ b/ci/images/alpine-tools/Dockerfile
@@ -36,10 +36,13 @@ RUN apk --no-cache add \
   && gcloud config set core/disable_usage_reporting true \
   && gcloud config set component_manager/disable_update_check true \
   && gcloud config set metrics/environment github_docker_image \
-  && git clone https://github.com/masterzen/winrm-cli \
-  && (cd winrm-cli; GOPATH=$PWD PATH=$PATH:$PWD/bin make) \
-  && cp winrm-cli/bin/winrm /usr/local/bin/ \
-  && rm -rf winrm-cli \
+  && mkdir -p gopath \
+  && cd gopath \
+  && export GOPATH=$(pwd) \
+  && go get github.com/masterzen/winrm-cli \
+  && cp ${GOPATH}/bin/winrm-cli /usr/local/bin/winrm \
+  && cd ${GOPATH}/.. \
+  && rm -rf ${GOPATH} \
   && gcloud components install -q beta \
   && printf "Host *\n  ServerAliveInterval 60 \n  ServerAliveCountMax 2\n" >> 
/etc/ssh/ssh_config \
   && pip2 install awscli \



[geode] branch release/1.12.0 updated: GEODE-7776 - Proposed changes to remove build cycle (#4679)

2020-02-12 Thread echobravo
This is an automated email from the ASF dual-hosted git repository.

echobravo pushed a commit to branch release/1.12.0
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/release/1.12.0 by this push:
 new fa8060d  GEODE-7776 - Proposed changes to remove build cycle (#4679)
fa8060d is described below

commit fa8060da4c7e5970f0ec538829c3f40de355b89f
Author: Robert Houghton 
AuthorDate: Mon Feb 10 13:18:34 2020 -0800

GEODE-7776 - Proposed changes to remove build cycle (#4679)

* Proposed changes to remove build cycle

* Fix expected pom, and delete commented dependency

(cherry picked from commit 7ab2713d6c9d2d09a986513aaa18b333951fe61e)
---
 geode-core/build.gradle|  4 
 geode-core/src/test/resources/expected-pom.xml | 12 
 2 files changed, 16 deletions(-)

diff --git a/geode-core/build.gradle b/geode-core/build.gradle
index ed80495..47da466 100755
--- a/geode-core/build.gradle
+++ b/geode-core/build.gradle
@@ -263,10 +263,6 @@ dependencies {
   //Log4j is used everywhere
   implementation('org.apache.logging.log4j:log4j-api')
 
-  runtimeOnly(project(':geode-gfsh')) {
-exclude module: 'geode-core'
-ext.optional = true
-  }
 
   runtimeOnly('io.swagger:swagger-annotations') {
 ext.optional = true
diff --git a/geode-core/src/test/resources/expected-pom.xml 
b/geode-core/src/test/resources/expected-pom.xml
index e2cdd14..6cb247a 100644
--- a/geode-core/src/test/resources/expected-pom.xml
+++ b/geode-core/src/test/resources/expected-pom.xml
@@ -243,18 +243,6 @@
   
 
 
-  org.apache.geode
-  geode-gfsh
-  runtime
-  
-
-  geode-core
-  *
-
-  
-  true
-
-
   io.swagger
   swagger-annotations
   runtime



[geode] 01/01: GEODE-7683: introduce BR.cmnClearRegion

2020-02-12 Thread nnag
This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch feature/GEODE-7665
in repository https://gitbox.apache.org/repos/asf/geode.git

commit d0daa20440449b4120015554cd7b10608f6e9e81
Author: zhouxh 
AuthorDate: Mon Jan 27 17:02:48 2020 -0800

GEODE-7683: introduce BR.cmnClearRegion
---
 .../apache/geode/internal/cache/BucketRegion.java  | 38 +--
 .../geode/internal/cache/DistributedRegion.java| 23 +--
 .../internal/cache/BucketRegionJUnitTest.java  | 77 ++
 .../internal/cache/DistributedRegionJUnitTest.java | 18 +
 4 files changed, 145 insertions(+), 11 deletions(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java 
b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
index ac20526..761c1b1 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
@@ -557,6 +557,36 @@ public class BucketRegion extends DistributedRegion 
implements Bucket {
 }
   }
 
+  @Override
+  void cmnClearRegion(RegionEventImpl regionEvent, boolean cacheWrite, boolean 
useRVV) {
+if (!getBucketAdvisor().isPrimary()) {
+  if (logger.isDebugEnabled()) {
+logger.debug("Not primary bucket when doing clear, do nothing");
+  }
+  return;
+}
+
+boolean enableRVV = useRVV && getConcurrencyChecksEnabled();
+RegionVersionVector rvv = null;
+if (enableRVV) {
+  rvv = getVersionVector().getCloneForTransmission();
+}
+
+// get rvvLock
+Set participants =
+getCacheDistributionAdvisor().adviseInvalidateRegion();
+try {
+  obtainWriteLocksForClear(regionEvent, participants);
+  // no need to dominate my own rvv.
+  // Clear is on going here, there won't be GII for this member
+  clearRegionLocally(regionEvent, cacheWrite, null);
+  distributeClearOperation(regionEvent, rvv, participants);
+
+  // TODO: call reindexUserDataRegion if there're lucene indexes
+} finally {
+  releaseWriteLocksForClear(regionEvent, participants);
+}
+  }
 
   long generateTailKey() {
 long key = 
eventSeqNum.addAndGet(partitionedRegion.getTotalNumberOfBuckets());
@@ -2094,11 +2124,9 @@ public class BucketRegion extends DistributedRegion 
implements Bucket {
   // if GII has failed, because there is not primary. So it's safe to set 
these
   // counters to 0.
   oldMemValue = bytesInMemory.getAndSet(0);
-}
-
-else {
-  throw new InternalGemFireError(
-  "Trying to clear a bucket region that was not destroyed or in 
initialization.");
+} else {
+  // BucketRegion's clear is supported now
+  oldMemValue = bytesInMemory.getAndSet(0);
 }
 if (oldMemValue != BUCKET_DESTROYED) {
   partitionedRegion.getPrStats().incDataStoreEntryCount(-sizeBeforeClear);
diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
 
b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
index 9964978..1a62919 100755
--- 
a/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java
@@ -2003,6 +2003,10 @@ public class DistributedRegion extends LocalRegion 
implements InternalDistribute
 super.basicClear(regionEvent, cacheWrite);
   }
 
+  void distributeClearOperation(RegionEventImpl regionEvent, 
RegionVersionVector rvv,
+  Set participants) {
+DistributedClearOperation.clear(regionEvent, rvv, participants);
+  }
 
   @Override
   void cmnClearRegion(RegionEventImpl regionEvent, boolean cacheWrite, boolean 
useRVV) {
@@ -2025,7 +2029,7 @@ public class DistributedRegion extends LocalRegion 
implements InternalDistribute
 obtainWriteLocksForClear(regionEvent, participants);
 clearRegionLocally(regionEvent, cacheWrite, null);
 if (!regionEvent.isOriginRemote() && 
regionEvent.getOperation().isDistributed()) {
-  DistributedClearOperation.clear(regionEvent, null, participants);
+  distributeClearOperation(regionEvent, null, participants);
 }
   } finally {
 releaseWriteLocksForClear(regionEvent, participants);
@@ -2081,10 +2085,12 @@ public class DistributedRegion extends LocalRegion 
implements InternalDistribute
   /**
* obtain locks preventing generation of new versions in other members
*/
-  private void obtainWriteLocksForClear(RegionEventImpl regionEvent,
+  protected void obtainWriteLocksForClear(RegionEventImpl regionEvent,
   Set participants) {
 lockLocallyForClear(getDistributionManager(), getMyId(), regionEvent);
-DistributedClearOperation.lockAndFlushToOthers(regionEvent, participants);
+if (!isUsedForPartitionedRegionBucket()) {
+  

[geode] branch feature/GEODE-7665 updated (8695340 -> d0daa20)

2020-02-12 Thread nnag
This is an automated email from the ASF dual-hosted git repository.

nnag pushed a change to branch feature/GEODE-7665
in repository https://gitbox.apache.org/repos/asf/geode.git.


omit 8695340  GEODE-7788: Alpine tools docker image creation fails due to 
internal (#4685)
omit c640394  GEODE-7683: introduce BR.cmnClearRegion
 add 5e8107a  GEODE-7788: Alpine tools docker image creation fails due to 
internal (#4685)
 add dcca237  GEODE-7760: fixing 
LocatorUDPSecurityDUnitTest.testCrashLocatorMultip… (#4683)
 add c5a3b16  GEODE-7792: configure logging for geode-membership 
integration tests (#4688)
 new d0daa20  GEODE-7683: introduce BR.cmnClearRegion

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (8695340)
\
 N -- N -- N   refs/heads/feature/GEODE-7665 (d0daa20)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../distributed/LocatorUDPSecurityDUnitTest.java |  1 +
 .../gms/messenger/GMSQuorumCheckerJUnitTest.java | 20 ++--
 .../gms/messenger/JGroupsMessengerJUnitTest.java |  2 +-
 .../src/integrationTest/resources/log4j2.xml | 12 ++--
 .../membership/gms/messenger/GMSEncrypt.java |  7 ---
 .../membership/gms/messenger/GMSQuorumChecker.java   |  8 ++--
 .../membership/gms/messenger/JGroupsMessenger.java   |  6 --
 .../gms/messenger/MembershipInformationImpl.java | 10 +-
 8 files changed, 41 insertions(+), 25 deletions(-)
 copy geode-log4j/src/main/resources/log4j2-cli.xml => 
geode-membership/src/integrationTest/resources/log4j2.xml (80%)
 mode change 100644 => 100755



[geode] branch develop updated (c5a3b16 -> d3ecaaa)

2020-02-12 Thread jensdeppe
This is an automated email from the ASF dual-hosted git repository.

jensdeppe pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.


from c5a3b16  GEODE-7792: configure logging for geode-membership 
integration tests (#4688)
 add d3ecaaa  GEODE-7778: Add PUBLISH, SUBSCRIBE and UNSUBSCRIBE Redis 
commands (#4682)

No new revisions were added by this update.

Summary of changes:
 .../org/apache/geode/redis/MockSubscriber.java |  35 ++-
 .../org/apache/geode/redis/PubSubDUnitTest.java|  91 
 .../java/org/apache/geode/redis/PubSubTest.java| 254 +
 .../apache/geode/redis/mocks/MockSubscriber.java   |  35 ++-
 .../org/apache/geode/redis/GeodeRedisServer.java   |  80 ---
 .../org/apache/geode/redis/internal/Client.java|  29 ++-
 .../org/apache/geode/redis/internal/Coder.java |  53 +++--
 .../redis/internal/ExecutionHandlerContext.java|  48 ++--
 .../org/apache/geode/redis/internal/GeoCoder.java  |   4 +-
 .../org/apache/geode/redis/internal/PubSub.java|  51 +
 .../apache/geode/redis/internal/PubSubImpl.java| 136 +++
 .../geode/redis/internal/RedisCommandType.java |  66 ++
 .../geode/redis/internal/RedisConstants.java   |   2 +
 .../apache/geode/redis/internal/RedisDataType.java |  14 +-
 .../apache/geode/redis/internal/Subscriber.java|  83 +++
 .../redis/internal/executor/AbstractExecutor.java  |   2 +-
 .../PublishExecutor.java}  |  36 +--
 .../executor/pubsub/SubscribeExecutor.java |  67 ++
 .../UnsubscribeExecutor.java}  |  49 ++--
 .../sanctioned-geode-redis-serializables.txt   |   5 +
 20 files changed, 983 insertions(+), 157 deletions(-)
 copy 
geode-gfsh/src/main/java/org/apache/geode/management/internal/cli/result/model/InfoResultModel.java
 => 
geode-redis/src/distributedTest/java/org/apache/geode/redis/MockSubscriber.java 
(58%)
 create mode 100644 
geode-redis/src/distributedTest/java/org/apache/geode/redis/PubSubDUnitTest.java
 create mode 100644 
geode-redis/src/integrationTest/java/org/apache/geode/redis/PubSubTest.java
 copy 
geode-gfsh/src/main/java/org/apache/geode/management/internal/cli/result/model/InfoResultModel.java
 => 
geode-redis/src/integrationTest/java/org/apache/geode/redis/mocks/MockSubscriber.java
 (58%)
 copy 
geode-management/src/main/java/org/apache/geode/management/runtime/IndexInfo.java
 => geode-redis/src/main/java/org/apache/geode/redis/internal/Client.java (64%)
 create mode 100644 
geode-redis/src/main/java/org/apache/geode/redis/internal/PubSub.java
 create mode 100644 
geode-redis/src/main/java/org/apache/geode/redis/internal/PubSubImpl.java
 create mode 100644 
geode-redis/src/main/java/org/apache/geode/redis/internal/Subscriber.java
 copy 
geode-redis/src/main/java/org/apache/geode/redis/internal/executor/{PersistExecutor.java
 => pubsub/PublishExecutor.java} (61%)
 mode change 100755 => 100644
 create mode 100755 
geode-redis/src/main/java/org/apache/geode/redis/internal/executor/pubsub/SubscribeExecutor.java
 copy 
geode-redis/src/main/java/org/apache/geode/redis/internal/executor/{TimeExecutor.java
 => pubsub/UnsubscribeExecutor.java} (53%)



[geode-native] branch develop updated: GEODE-7794: Fix clangformat issue

2020-02-12 Thread bbender
This is an automated email from the ASF dual-hosted git repository.

bbender pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
 new 1829f0e  GEODE-7794: Fix clangformat issue
1829f0e is described below

commit 1829f0e88748fd365bcbb4b091aea3bd50f36174
Author: Blake Bender 
AuthorDate: Wed Feb 12 11:43:22 2020 -0800

GEODE-7794: Fix clangformat issue
---
 cppcache/src/PdxType.hpp | 9 -
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/cppcache/src/PdxType.hpp b/cppcache/src/PdxType.hpp
index 0ed16b0..b76b824 100644
--- a/cppcache/src/PdxType.hpp
+++ b/cppcache/src/PdxType.hpp
@@ -43,11 +43,10 @@ typedef std::map> NameVsPdxType;
 class PdxType;
 class PdxTypeRegistry;
 
-class PdxType
-: public internal::DataSerializableInternal,
-  public std::enable_shared_from_this,
-  private NonCopyable,
-  private NonAssignable {
+class PdxType : public internal::DataSerializableInternal,
+public std::enable_shared_from_this,
+private NonCopyable,
+private NonAssignable {
  private:
   ACE_RW_Thread_Mutex m_lockObj;
 



[geode-native] branch develop updated: GEODE-7794: Checking for null result and falling back to PdxInstance deserialization. (#575)

2020-02-12 Thread bbender
This is an automated email from the ASF dual-hosted git repository.

bbender pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
 new 728c393  GEODE-7794: Checking for null result and falling back to 
PdxInstance deserialization. (#575)
728c393 is described below

commit 728c393150c2a83dad2ba00318561844520de55e
Author: Matthew Reddington 
AuthorDate: Wed Feb 12 14:14:41 2020 -0800

GEODE-7794: Checking for null result and falling back to PdxInstance 
deserialization. (#575)

Fixes a bug where, when issuing two get() requests back-to-back for the 
same key, the second result returned is a PdxWrapper object with a shared 
pointer to null.
---
 cppcache/integration/test/PdxJsonTypeTest.cpp | 29 +++
 cppcache/src/PdxHelper.cpp|  9 +
 cppcache/src/PdxType.hpp  |  9 +
 3 files changed, 43 insertions(+), 4 deletions(-)

diff --git a/cppcache/integration/test/PdxJsonTypeTest.cpp 
b/cppcache/integration/test/PdxJsonTypeTest.cpp
index e98eb48..33643dc 100644
--- a/cppcache/integration/test/PdxJsonTypeTest.cpp
+++ b/cppcache/integration/test/PdxJsonTypeTest.cpp
@@ -113,4 +113,33 @@ TEST(PdxJsonTypeTest, testCreateTwoJsonInstances) {
   EXPECT_EQ(pdxInstance->getIntField("baz"), 42);
 }
 
+TEST(PdxJsonTypeTest, testTwoConsecutiveGets) {
+  Cluster cluster{LocatorCount{1}, ServerCount{1}};
+  cluster.start();
+  cluster.getGfsh()
+  .create()
+  .region()
+  .withName("region")
+  .withType("REPLICATE")
+  .execute();
+
+  auto cache = cluster.createCache();
+  auto region = setupRegion(cache);
+  auto pdxInstanceFactory =
+  cache.createPdxInstanceFactory(gemfireJsonClassName);
+
+  pdxInstanceFactory.writeString("foo", "bar");
+  auto pdxInstance = pdxInstanceFactory.create();
+
+  region->put("simpleObject", pdxInstance);
+
+  auto cache2 = cluster.createCache();
+  auto region2 = setupRegion(cache2);
+
+  region2->get("simpleObject");  // Throw the first one away
+
+  EXPECT_TRUE(
+  std::dynamic_pointer_cast(region2->get("simpleObject")));
+}
+
 }  // namespace
diff --git a/cppcache/src/PdxHelper.cpp b/cppcache/src/PdxHelper.cpp
index fd5efdf..806b8d3 100644
--- a/cppcache/src/PdxHelper.cpp
+++ b/cppcache/src/PdxHelper.cpp
@@ -176,6 +176,15 @@ std::shared_ptr 
PdxHelper::deserializePdx(DataInput& dataInput,
 ->getExpiryTaskManager());  // it will set data in weakhashmap
   }
   prr.moveStream();
+
+  if (auto pdxWrapper =
+  std::dynamic_pointer_cast(pdxObjectptr)) {
+if (!pdxWrapper->getObject()) {
+  // No serializer was registered to deserialize this type.
+  // Fall back to PdxInstance
+  return nullptr;
+}
+  }
 }
   } else {
 // type not found; need to get from server
diff --git a/cppcache/src/PdxType.hpp b/cppcache/src/PdxType.hpp
index b76b824..0ed16b0 100644
--- a/cppcache/src/PdxType.hpp
+++ b/cppcache/src/PdxType.hpp
@@ -43,10 +43,11 @@ typedef std::map> NameVsPdxType;
 class PdxType;
 class PdxTypeRegistry;
 
-class PdxType : public internal::DataSerializableInternal,
-public std::enable_shared_from_this,
-private NonCopyable,
-private NonAssignable {
+class PdxType
+: public internal::DataSerializableInternal,
+  public std::enable_shared_from_this,
+  private NonCopyable,
+  private NonAssignable {
  private:
   ACE_RW_Thread_Mutex m_lockObj;
 



[geode] branch develop updated (d3ecaaa -> 1854423)

2020-02-12 Thread bschuchardt
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.


from d3ecaaa  GEODE-7778: Add PUBLISH, SUBSCRIBE and UNSUBSCRIBE Redis 
commands (#4682)
 add 1854423  GEODE-7796: CI failure in LocatorDUnitTest (#4693)

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/geode/distributed/LocatorDUnitTest.java | 2 ++
 1 file changed, 2 insertions(+)



[geode] branch feature/GEODE-7746 updated (606a8ea -> 0303da6)

2020-02-12 Thread jchen21
This is an automated email from the ASF dual-hosted git repository.

jchen21 pushed a change to branch feature/GEODE-7746
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 606a8ea  Code formatting
 add 0303da6  Remove the DUnit test

No new revisions were added by this update.

Summary of changes:
 .../util/ClientServerSessionCacheDUnitTest.java|  3 +-
 .../cache/execute/FunctionExecutionDUnit.java  | 96 --
 .../cache/client/internal/ExecuteFunctionOp.java   | 19 ++---
 3 files changed, 10 insertions(+), 108 deletions(-)
 delete mode 100644 
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/execute/FunctionExecutionDUnit.java



[geode] branch feature/GEODE-7796 created (now 0e21a58)

2020-02-12 Thread bschuchardt
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a change to branch feature/GEODE-7796
in repository https://gitbox.apache.org/repos/asf/geode.git.


  at 0e21a58  GEODE-7796: CI failure in LocatorDUnitTest

This branch includes the following new commits:

 new 0e21a58  GEODE-7796: CI failure in LocatorDUnitTest

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[geode] 01/01: GEODE-7796: CI failure in LocatorDUnitTest

2020-02-12 Thread bschuchardt
This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a commit to branch feature/GEODE-7796
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 0e21a585e77fa531fe5279dc789cf41efe29a923
Author: Bruce Schuchardt 
AuthorDate: Wed Feb 12 15:22:57 2020 -0800

GEODE-7796: CI failure in LocatorDUnitTest

The new test hung during a CI run due to a flaw in Geode.  I'm disabling
it until the flaw is corrected.
---
 .../java/org/apache/geode/distributed/LocatorDUnitTest.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
 
b/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
index 4771ff3..d6eb991 100644
--- 
a/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
+++ 
b/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
@@ -72,6 +72,7 @@ import java.util.Set;
 import org.apache.logging.log4j.Logger;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
@@ -255,6 +256,7 @@ public class LocatorDUnitTest implements Serializable {
   }
 
   @Test
+  @Ignore("GEODE=7760 - test sometimes hangs due to product issue")
   public void testCrashLocatorMultipleTimes() throws Exception {
 port1 = AvailablePort.getRandomAvailablePort(AvailablePort.SOCKET);
 DistributedTestUtils.deleteLocatorStateFile(port1);



[geode] branch feature/GEODE-7746 updated (0303da6 -> 7768349)

2020-02-12 Thread jchen21
This is an automated email from the ASF dual-hosted git repository.

jchen21 pushed a change to branch feature/GEODE-7746
in repository https://gitbox.apache.org/repos/asf/geode.git.


 discard 0303da6  Remove the DUnit test
 add 7768349  Remove the DUnit test

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (0303da6)
\
 N -- N -- N   refs/heads/feature/GEODE-7746 (7768349)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .../org/apache/geode/modules/util/ClientServerSessionCacheDUnitTest.java | 1 -
 1 file changed, 1 deletion(-)