[geode] branch develop updated (86b5f7b -> 7c01027)

2022-03-29 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 86b5f7b  GEODE-10127: Adds test to preserve correct marshalling 
behavior. (#7461)
 add 7c01027  GEODE-10190: Improve runntime of MSetIntegrationTest and 
RenameIntegratinoTest (#7511)

No new revisions were added by this update.

Summary of changes:
 .../commands/executor/key/AbstractRenameIntegrationTest.java| 6 +++---
 .../commands/executor/string/AbstractMSetIntegrationTest.java   | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)


[geode] 06/06: GEODE-9980: Improve error handling of serial filters (#7299)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 9fc67eda1e6b64ab30548e97648efbb8440ab883
Author: Kirk Lund 
AuthorDate: Tue Feb 8 12:04:44 2022 -0800

GEODE-9980: Improve error handling of serial filters (#7299)

Improves the error handling of serial filter configuration and
filtering. The API throws a runtime exception wrapping any thrown
exceptions.

When geode.enableGlobalSerialFilter is FALSE:
* Startup succeeds without throwing any exceptions even if an older
  version of Java throws "java.lang.ClassNotFoundException
  sun.misc.ObjectInputFilter".

When geode.enableGlobalSerialFilter is TRUE:
* Startup fails by throwing an exception when configuration of serial
  filter fails for any reason.

Renames ObjectInputFilter to avoid confusion with the Java interface of
the same name.

Fixes a bug found in ReflectiveFacadeGlobalSerialFilterTest which
resulted in configuring a non-mocked process-wide serial filter that
polluted the JVM for all later tests.

Fixes other minor details in LocatorLauncher, InternalDataSerializer
and various related tests.

(cherry picked from commit ce57e9fd2b8b644cadc469209e12e4fbd281e0d9)
---
 .../filter/SerialFilterAssertions.java |  30 +++---
 .../apache/geode/distributed/LocatorLauncher.java  |  16 ++-
 .../apache/geode/distributed/ServerLauncher.java   |  16 ++-
 .../geode/internal/InternalDataSerializer.java |  29 --
 .../geode/management/internal/ManagementAgent.java |   7 +-
 .../geode/distributed/LocatorLauncherTest.java |   2 +-
 ...ationWhenFilterIsAlreadySetIntegrationTest.java |  53 ++
 ...enObjectInputFilterNotFoundIntegrationTest.java | 108 
 ...nputFilterApiSetFilterBlankIntegrationTest.java |   1 -
 java => FilterAlreadyConfiguredException.java} |  22 +++--
 .../serialization/filter/FilterConfiguration.java  |   2 +-
 .../serialization/filter/GlobalSerialFilter.java   |   5 +-
 .../filter/GlobalSerialFilterConfiguration.java|  85 ++--
 .../filter/JmxSerialFilterConfiguration.java   |  15 +--
 ...nputFilter.java => NullStreamSerialFilter.java} |   4 +-
 .../filter/ObjectInputFilterInvocationHandler.java | 109 +
 .../filter/ReflectiveFacadeGlobalSerialFilter.java |  43 ++--
 .../ReflectiveFacadeGlobalSerialFilterFactory.java |  20 ++--
 .../ReflectiveFacadeObjectInputFilterFactory.java  |  61 
 ...ava => ReflectiveFacadeStreamSerialFilter.java} |  49 +++--
 ...ReflectiveFacadeStreamSerialFilterFactory.java} |  35 +++
 .../filter/ReflectiveObjectInputFilterApi.java |  59 +++
 ...ectInputFilter.java => StreamSerialFilter.java} |   7 +-
 ...Factory.java => StreamSerialFilterFactory.java} |   4 +-
 ...ertyGlobalSerialFilterConfigurationFactory.java |  24 -
 java => UnableToSetSerialFilterException.java} |  22 +++--
 ...anctioned-geode-serialization-serializables.txt |   2 +
 .../GlobalSerialFilterConfigurationTest.java   |  84 
 .../JmxSerialFilterConfigurationFactoryTest.java   |   9 ++
 .../filter/JmxSerialFilterConfigurationTest.java   |  50 ++
 .../filter/NullObjectInputFilterTest.java  |   4 +-
 .../ObjectInputFilterInvocationHandlerTest.java|  97 ++
 ...lectiveFacadeGlobalSerialFilterFactoryTest.java |  65 ++--
 .../ReflectiveFacadeGlobalSerialFilterTest.java|  88 ++---
 ...flectiveFacadeObjectInputFilterFactoryTest.java |  12 +--
 .../ReflectiveFacadeObjectInputFilterTest.java |  77 ++-
 .../ReflectiveObjectInputFilterApiFactoryTest.java |   9 ++
 .../filter/ReflectiveObjectInputFilterApiTest.java |  11 ++-
 .../filter/SerialFilterAssertions.java |  53 +-
 ...GlobalSerialFilterConfigurationFactoryTest.java | 101 ---
 ...rtyJmxSerialFilterConfigurationFactoryTest.java |  12 ++-
 41 files changed, 1067 insertions(+), 435 deletions(-)

diff --git 
a/geode-core/src/integrationTest/java/org/apache/geode/internal/serialization/filter/SerialFilterAssertions.java
 
b/geode-core/src/integrationTest/java/org/apache/geode/internal/serialization/filter/SerialFilterAssertions.java
index b6d552a..8432e7d 100644
--- 
a/geode-core/src/integrationTest/java/org/apache/geode/internal/serialization/filter/SerialFilterAssertions.java
+++ 
b/geode-core/src/integrationTest/java/org/apache/geode/internal/serialization/filter/SerialFilterAssertions.java
@@ -30,35 +30,29 @@ public class SerialFilterAssertions {
 
   public static void assertThatSerialFilterIsNull()
   throws InvocationTargetException, IllegalAccessException {
-boolean exists = API.getSerialFilter() != null;
-assertThat(exists)
-.as("ObjectInputFilter$Config.getSerialFilter() is null")
-

[geode] 04/06: GEODE-9879: Extract SystemProperty to geode-common (#7177)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit a8d932b286f94ec15464d179d860df048375f09b
Author: Kirk Lund 
AuthorDate: Mon Dec 20 10:53:32 2021 -0800

GEODE-9879: Extract SystemProperty to geode-common (#7177)

Extract part of SystemPropertyHelper to geode-common for use in
modules that are upstream from geode-core.

Define new DEFAULT_PREFIX that maps to GEODE_PREFIX and use that
everywhere that GEODE_PREFIX was previously used.

* Inline prefix constants in SystemPropertyHelper
* Use static imports for system property prefixes.
* Split up large tests in SystemPropertyHelperTest.
* Fix JAXBService

(cherry picked from commit ebf8479fffb5775b1f45801aa9382c2dad7106e0)
---
 geode-common/build.gradle  |   3 +
 .../apache/geode/internal/lang/SystemProperty.java | 118 
 .../geode/internal/lang/SystemPropertyTest.java| 107 +++
 ...okenSerializationConsistencyRegressionTest.java |   6 +-
 ...entDeserializationCopyOnReadRegressionTest.java |   7 +-
 .../cache/CompactOfflineDiskStoreDUnitTest.java|   5 +-
 .../internal/cache/DeltaPropagationDUnitTest.java  |   2 +-
 .../cache/ParallelDiskStoreRecoveryDUnitTest.java  |   5 +-
 .../cache/PartitionedRegionSingleHopDUnitTest.java |   2 +-
 .../cache/ValidateOfflineDiskStoreDUnitTest.java   |   5 +-
 .../RebalanceOperationComplexDistributedTest.java  |   4 +-
 .../cache/ha/HARQueueNewImplDUnitTest.java |   5 +-
 .../ha/HARegionQueueExpiryRegressionTest.java  |   5 +-
 .../HARegionQueueThreadIdExpiryRegressionTest.java |   8 +-
 .../sockets/RedundancyLevelPart1DUnitTest.java |   2 +-
 .../internal/cache/ha/HARegionQueueJUnitTest.java  |   4 +-
 .../DefaultDiskDirsIntegrationTest.java|   4 +-
 .../internal/cache/EntryEventSerialization.java|   2 +-
 .../cache/eviction/EvictionListBuilder.java|   3 +-
 .../cache/eviction/LRUListWithAsyncSorting.java|   5 +-
 .../cache/eviction/LRUListWithSyncSorting.java |   3 +-
 .../geode/internal/cache/ha/HARegionQueue.java |   2 +-
 .../cache/persistence/DefaultDiskDirs.java |   2 +-
 .../internal/cache/xmlcache/CacheCreation.java |   3 +-
 .../apache/geode/internal/config/JAXBService.java  |   6 +-
 .../geode/internal/lang/SystemPropertyHelper.java  |  89 +
 .../internal/logging/CoreLoggingExecutors.java |   2 +-
 .../geode/logging/internal/Configuration.java  |   6 +-
 .../logging/internal/LoggingProviderLoader.java|   5 +-
 .../cache/EntryEventSerializationTest.java |   4 +-
 .../cache/persistence/DefaultDiskDirsTest.java |   6 +-
 .../PersistenceInitialImageAdvisorTest.java|   5 +-
 .../internal/lang/SystemPropertyHelperTest.java| 148 -
 .../DistributedDiskDirRuleDistributedTest.java |  17 ++-
 .../junit/rules/DiskDirRuleIntegrationTest.java|   5 +-
 .../test/dunit/rules/DistributedDiskDirRule.java   |  16 ++-
 .../apache/geode/test/junit/rules/DiskDirRule.java |  14 +-
 37 files changed, 411 insertions(+), 224 deletions(-)

diff --git a/geode-common/build.gradle b/geode-common/build.gradle
index 7506af7..ab33aff 100755
--- a/geode-common/build.gradle
+++ b/geode-common/build.gradle
@@ -27,6 +27,9 @@ dependencies {
   implementation('com.fasterxml.jackson.core:jackson-databind')
 
   // test
+  testImplementation('com.github.stefanbirkner:system-rules') {
+exclude module: 'junit-dep'
+  }
   testImplementation('junit:junit')
   testImplementation('org.apache.commons:commons-lang3')
   testImplementation('org.assertj:assertj-core')
diff --git 
a/geode-common/src/main/java/org/apache/geode/internal/lang/SystemProperty.java 
b/geode-common/src/main/java/org/apache/geode/internal/lang/SystemProperty.java
new file mode 100644
index 000..2c433ef
--- /dev/null
+++ 
b/geode-common/src/main/java/org/apache/geode/internal/lang/SystemProperty.java
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more 
contributor license
+ * agreements. See the NOTICE file distributed with this work for additional 
information regarding
+ * copyright ownership. The ASF licenses this file to You under the Apache 
License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the 
License. You may obtain a
+ * copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software 
distributed under the License
+ * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY 
KIND, either express
+ * or implied. See the License for the specific language governing permissions 
and limitations under
+ * the License.
+ */
+package org.apache.geode.internal.lang;
+
+import java.util.Optional;
+
+public class 

[geode] 03/06: GEODE-9758: Move ClassUtils to geode-common (#7166)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit cc227bc894324d3e65cf06f085c645f063df8635
Author: Kirk Lund 
AuthorDate: Fri Dec 10 11:28:17 2021 -0800

GEODE-9758: Move ClassUtils to geode-common (#7166)

(cherry picked from commit b6fca291378a1bc334b0f9927c899a1892442939)
---
 .../src/main/java/org/apache/geode/internal/lang/ClassUtils.java  | 2 +-
 .../java/org/apache/geode/internal/lang/utils/ClassUtilsTest.java | 8 
 .../main/java/org/apache/geode/distributed/AbstractLauncher.java  | 2 +-
 .../java/org/apache/geode/internal/InternalDataSerializer.java| 2 +-
 .../InternalDataSerializerSerializationAcceptlistTest.java| 2 +-
 .../geode/internal/InternalDataSerializerShiroAcceptListTest.java | 2 +-
 .../internal/cli/commands/DescribeDiskStoreCommand.java   | 2 +-
 .../java/org/apache/geode/management/internal/cli/shell/Gfsh.java | 2 +-
 .../java/org/apache/geode/codeAnalysis/CompiledClassUtils.java| 8 +---
 .../internal/serialization/SerializationDependenciesTest.java | 3 ++-
 .../java/org/apache/geode/rest/internal/web/util/NumberUtils.java | 2 +-
 11 files changed, 19 insertions(+), 16 deletions(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/lang/ClassUtils.java 
b/geode-common/src/main/java/org/apache/geode/internal/lang/ClassUtils.java
similarity index 99%
rename from 
geode-core/src/main/java/org/apache/geode/internal/lang/ClassUtils.java
rename to 
geode-common/src/main/java/org/apache/geode/internal/lang/ClassUtils.java
index 82bae79..82198dd 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/lang/ClassUtils.java
+++ b/geode-common/src/main/java/org/apache/geode/internal/lang/ClassUtils.java
@@ -13,7 +13,7 @@
  * the License.
  */
 
-package org.apache.geode.internal.lang;
+package org.apache.geode.internal.lang.utils;
 
 import org.apache.geode.annotations.Immutable;
 
diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/lang/ClassUtilsJUnitTest.java
 
b/geode-common/src/test/java/org/apache/geode/internal/lang/utils/ClassUtilsTest.java
similarity index 97%
rename from 
geode-core/src/test/java/org/apache/geode/internal/lang/ClassUtilsJUnitTest.java
rename to 
geode-common/src/test/java/org/apache/geode/internal/lang/utils/ClassUtilsTest.java
index ad118ef..49e3714 100644
--- 
a/geode-core/src/test/java/org/apache/geode/internal/lang/ClassUtilsJUnitTest.java
+++ 
b/geode-common/src/test/java/org/apache/geode/internal/lang/utils/ClassUtilsTest.java
@@ -12,7 +12,7 @@
  * or implied. See the License for the specific language governing permissions 
and limitations under
  * the License.
  */
-package org.apache.geode.internal.lang;
+package org.apache.geode.internal.lang.utils;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
@@ -22,20 +22,20 @@ import static org.junit.Assert.assertTrue;
 import java.util.Calendar;
 import java.util.Date;
 
+import org.apache.commons.lang3.StringUtils;
 import org.junit.Test;
 
-
 /**
  * The ClassUtilsJUnitTest class is a test suite with test cases to test the 
contract and
  * functionality of the ClassUtils class.
  * 
  *
- * @see org.apache.geode.internal.lang.ClassUtils
+ * @see ClassUtils
  * @see org.junit.Assert
  * @see org.junit.Test
  * @since GemFire 7.0
  */
-public class ClassUtilsJUnitTest {
+public class ClassUtilsTest {
 
   @Test
   public void testForNameWithExistingClass() {
diff --git 
a/geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java 
b/geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java
index 6734666..270b2a5 100644
--- 
a/geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java
+++ 
b/geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java
@@ -18,9 +18,9 @@ import static org.apache.commons.lang3.StringUtils.isNotBlank;
 import static org.apache.commons.lang3.StringUtils.join;
 import static org.apache.commons.lang3.StringUtils.lowerCase;
 import static org.apache.geode.distributed.ConfigurationProperties.NAME;
-import static org.apache.geode.internal.lang.ClassUtils.forName;
 import static org.apache.geode.internal.lang.StringUtils.defaultString;
 import static org.apache.geode.internal.lang.SystemUtils.CURRENT_DIRECTORY;
+import static org.apache.geode.internal.lang.utils.ClassUtils.forName;
 
 import java.io.File;
 import java.io.FileReader;
diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/InternalDataSerializer.java
 
b/geode-core/src/main/java/org/apache/geode/internal/InternalDataSerializer.java
index cba2560..9c5018a 100644
--- 
a/geode-core/src/main/java/org/apache/geode/internal/InternalDataSerializer.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/InternalDataSerializer.java
@@ -103,7 +103,7 @@ import 

[geode] 02/06: GEODE-9758: Move SanctionedSerializables to filter package (#7165)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 6d0a4f10a5d75811d064726f57e19c15c47ad0b5
Author: Kirk Lund 
AuthorDate: Thu Dec 9 10:15:20 2021 -0800

GEODE-9758: Move SanctionedSerializables to filter package (#7165)

Move SanctionedSerializables to new package
org.apache.geode.internal.serialization.filter.

(cherry picked from commit db64b4948e790d61e82f95ae6163a62adc4c67fb)
---
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../RedisSanctionedSerializablesService.java   |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 geode-common/build.gradle  |  1 +
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../ConnectorsSanctionedSerializablesService.java  |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 geode-core/build.gradle|  2 +
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../CoreSanctionedSerializablesService.java|  2 +-
 .../geode/internal/InternalDataSerializer.java |  4 +-
 .../internal/ObjectInputStreamFilterWrapper.java   |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../internal/CQSanctionedSerializablesService.java |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../DUnitSanctionedSerializablesService.java   |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 .../AnalyzeGfshSerializablesIntegrationTest.java   |  2 +-
 ...ctionedSerializablesServiceIntegrationTest.java |  4 +-
 .../GfshSanctionedSerializablesService.java|  4 +-
 ...al.serialization.SanctionedSerializablesService | 15 
 ...alization.filter.SanctionedSerializablesService |  2 +-
 .../sanctioned-geode-gfsh-serializables.txt|  0
 ...ctionedSerializablesServiceIntegrationTest.java |  5 +-
 ...nedSerializablesServiceIntegrationTestBase.java | 16 ++--
 .../categories/SanctionedSerializablesTest.java|  5 +-
 .../JUnitSanctionedSerializablesService.java   |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../LuceneSanctionedSerializablesService.java  |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../ManagementSanctionedSerializablesService.java  |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 .../MembershipDependenciesJUnitTest.java   |  1 -
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../MembershipSanctionedSerializablesService.java  |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../MemcachedSanctionedSerializablesService.java   |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 geode-serialization/build.gradle   |  4 +
 ...ctionedSerializablesServiceIntegrationTest.java |  1 +
 .../apache/geode/codeAnalysis/excludedClasses.txt  |  0
 ...erializationSanctionedSerializablesService.java |  2 +
 .../{ => filter}/SanctionedSerializables.java  | 37 -
 .../SanctionedSerializablesService.java|  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 ...est.java => SerializationDependenciesTest.java} | 20 ++---
 .../filter/SanctionedSerializablesTest.java| 95 ++
 .../src/test/resources/expected-pom.xml| 10 +++
 ...ctionedSerializablesServiceIntegrationTest.java |  4 +-
 .../AnalyzeWANSerializablesIntegrationTest.java|  2 +-
 .../WANSanctionedSerializablesService.java |  4 +-
 ...lization.filter.SanctionedSerializablesService} |  2 +-
 .../sanctioned-geode-wan-serializables.txt |  0
 ...ctionedSerializablesServiceIntegrationTest.java |  2 +-
 .../WebApiSanctionedSerializablesService.java  |  2 +-
 ...lization.filter.SanctionedSerializablesService} |  0
 61 files changed, 213 insertions(+), 75 deletions(-)

diff --git 
a/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/RedisSanctionedSerializablesServiceIntegrationTest.java
 
b/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/RedisSanctionedSerializablesServiceIntegrationTest.java
index 420db71..82f8cf9 100644
--- 
a/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/RedisSanctionedSerializablesServiceIntegrationTest.java
+++ 
b/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/RedisSanctionedSerializablesServiceIntegrationTest.java
@@ -17,7 +17,7 @@ package org.apache.geode.redis.internal;
 import org.junit.experimental.categories.Category;

[geode] 01/06: GEODE-9817: Enable customized source set paths for ClassAnalysisRule (#7121)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 31375b759fcfa91d7df9749f3ffc556fa57f9d65
Author: Kirk Lund 
AuthorDate: Thu Nov 18 16:23:36 2021 -0800

GEODE-9817: Enable customized source set paths for ClassAnalysisRule (#7121)

Adds support for customizing source set paths of ClassAnalysisRule.

PROBLEM

Modules external to Geode must be structured the same as Geode
source code in order to use ClassAnalysisRule and the
Analyze*Serializables tests. This is necessary to better facilitate
pluggability of modules that need to provide sanctioned serializable
lists.

SOLUTION

Add source set path customization to ClassAnalysisRule, introduce
a new layer of Analyze*Serializables test base classes that can be
directly extended in order to customize source set paths in
ClassAnalysisRule. Also includes improvements to some iterating
of classes during analysis.

(cherry picked from commit 5d1e91932dff296632916a6ceccfb36039357acd)
---
 .../AnalyzeRedisSerializablesIntegrationTest.java  |  3 +-
 ...lyzeConnectorsSerializablesIntegrationTest.java |  2 +-
 .../AnalyzeCoreSerializablesIntegrationTest.java   |  3 +-
 .../AnalyzeCQSerializablesIntegrationTest.java |  3 +-
 .../AnalyzeDUnitSerializablesIntegrationTest.java  |  4 +-
 .../AnalyzeGfshSerializablesIntegrationTest.java   |  3 +-
 .../AnalyzeJUnitSerializablesIntegrationTest.java  |  3 +-
 java => AnalyzeDataSerializablesTestBase.java} | 74 ++
 ...SerializablesWithClassAnalysisRuleTestBase.java | 26 
 ...Base.java => AnalyzeSerializablesTestBase.java} | 41 
 ...SerializablesWithClassAnalysisRuleTestBase.java | 23 ---
 ...nedSerializablesServiceIntegrationTestBase.java |  6 +-
 .../geode/test/junit/rules/ClassAnalysisRule.java  | 23 ---
 .../AnalyzeLuceneSerializablesIntegrationTest.java |  3 +-
 ...lyzeManagementSerializablesIntegrationTest.java |  2 +-
 ...lyzeMembershipSerializablesIntegrationTest.java |  2 +-
 ...alyzeMemcachedSerializablesIntegrationTest.java |  2 +-
 .../AnalyzePulseSerializablesIntegrationTest.java  |  3 +-
 ...eSerializationSerializablesIntegrationTest.java |  2 +-
 .../AnalyzeWANSerializablesIntegrationTest.java|  3 +-
 .../AnalyzeWebApiSerializablesIntegrationTest.java |  3 +-
 21 files changed, 134 insertions(+), 100 deletions(-)

diff --git 
a/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeRedisSerializablesIntegrationTest.java
 
b/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeRedisSerializablesIntegrationTest.java
index 7c89c5b..36efe19 100755
--- 
a/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeRedisSerializablesIntegrationTest.java
+++ 
b/geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeRedisSerializablesIntegrationTest.java
@@ -22,7 +22,8 @@ import 
org.apache.geode.redis.internal.RedisSanctionedSerializablesService;
 import org.apache.geode.test.junit.categories.SerializationTest;
 
 @Category(SerializationTest.class)
-public class AnalyzeRedisSerializablesIntegrationTest extends 
AnalyzeSerializablesJUnitTestBase {
+public class AnalyzeRedisSerializablesIntegrationTest
+extends AnalyzeSerializablesWithClassAnalysisRuleTestBase {
 
   @Override
   protected String getModuleName() {
diff --git 
a/geode-connectors/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeConnectorsSerializablesIntegrationTest.java
 
b/geode-connectors/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeConnectorsSerializablesIntegrationTest.java
index a5fa5a1..db50c2f 100644
--- 
a/geode-connectors/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeConnectorsSerializablesIntegrationTest.java
+++ 
b/geode-connectors/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeConnectorsSerializablesIntegrationTest.java
@@ -23,7 +23,7 @@ import 
org.apache.geode.test.junit.categories.SerializationTest;
 
 @Category(SerializationTest.class)
 public class AnalyzeConnectorsSerializablesIntegrationTest
-extends AnalyzeSerializablesJUnitTestBase {
+extends AnalyzeSerializablesWithClassAnalysisRuleTestBase {
 
   @Override
   protected String getModuleName() {
diff --git 
a/geode-core/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeCoreSerializablesIntegrationTest.java
 
b/geode-core/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeCoreSerializablesIntegrationTest.java
index 6832442..d93d8d1 100755
--- 
a/geode-core/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeCoreSerializablesIntegrationTest.java
+++ 
b/geode-core/src/integrationTest/java/org/apache/geode/codeAnalysis/AnalyzeCoreSerializablesIntegrationTest.java

[geode] branch support/1.14 updated (3f62606 -> 9fc67ed)

2022-03-29 Thread klund
This is an automated email from the ASF dual-hosted git repository.

klund pushed a change to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 3f62606  GEODE-10179: Bump jackson-databind to 2.12.6.1 (#7501)
 new 31375b7  GEODE-9817: Enable customized source set paths for 
ClassAnalysisRule (#7121)
 new 6d0a4f1  GEODE-9758: Move SanctionedSerializables to filter package 
(#7165)
 new cc227bc  GEODE-9758: Move ClassUtils to geode-common (#7166)
 new a8d932b  GEODE-9879: Extract SystemProperty to geode-common (#7177)
 new 92e0e89  GEODE-9758: Add internal serial filter API (#7217)
 new 9fc67ed  GEODE-9980: Improve error handling of serial filters (#7299)

The 6 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:
 .../AnalyzeRedisSerializablesIntegrationTest.java  |   3 +-
 ...ctionedSerializablesServiceIntegrationTest.java |   2 +-
 .../RedisSanctionedSerializablesService.java   |   2 +-
 ...lization.filter.SanctionedSerializablesService} |   0
 ...artLocatorGlobalSerialFilterAcceptanceTest.java | 138 
 ...StartLocatorJmxSerialFilterAcceptanceTest.java} |  58 ++--
 ...tartServerGlobalSerialFilterAcceptanceTest.java | 150 +
 .../StartServerJmxSerialFilterAcceptanceTest.java} |  55 ++--
 geode-common/build.gradle  |   4 +
 .../org/apache/geode/internal/lang/ClassUtils.java |   2 +-
 .../apache/geode/internal/lang/SystemProperty.java | 118 +++
 .../geode/internal/lang/SystemPropertyTest.java| 107 +++
 .../geode/internal/lang/utils/ClassUtilsTest.java  |   8 +-
 ...lyzeConnectorsSerializablesIntegrationTest.java |   2 +-
 ...ctionedSerializablesServiceIntegrationTest.java |   2 +-
 .../ConnectorsSanctionedSerializablesService.java  |   2 +-
 ...lization.filter.SanctionedSerializablesService} |   0
 geode-core/build.gradle|   4 +-
 ...ValidateSerializableObjectsDistributedTest.java | 320 +++
 ...okenSerializationConsistencyRegressionTest.java |   6 +-
 ...entDeserializationCopyOnReadRegressionTest.java |   7 +-
 .../cache/CompactOfflineDiskStoreDUnitTest.java|   5 +-
 .../internal/cache/DeltaPropagationDUnitTest.java  |   2 +-
 .../cache/ParallelDiskStoreRecoveryDUnitTest.java  |   5 +-
 .../cache/PartitionedRegionSingleHopDUnitTest.java |   2 +-
 .../cache/ValidateOfflineDiskStoreDUnitTest.java   |   5 +-
 .../RebalanceOperationComplexDistributedTest.java  |   4 +-
 .../cache/ha/HARQueueNewImplDUnitTest.java |   5 +-
 .../ha/HARegionQueueExpiryRegressionTest.java  |   5 +-
 .../HARegionQueueThreadIdExpiryRegressionTest.java |   8 +-
 .../sockets/RedundancyLevelPart1DUnitTest.java |   2 +-
 .../AnalyzeCoreSerializablesIntegrationTest.java   |   3 +-
 .../geode/distributed/LocatorIntegrationTest.java  | 180 ++-
 ...catorLauncherJmxManagerLocalRegressionTest.java |  22 +-
 ...ctionedSerializablesServiceIntegrationTest.java |   2 +-
 .../internal/cache/ha/HARegionQueueJUnitTest.java  |   4 +-
 .../DefaultDiskDirsIntegrationTest.java|   4 +-
 ...cherGlobalSerialFilterBlankIntegrationTest.java |  55 
 ...herGlobalSerialFilterExistsIntegrationTest.java |  55 
 ...rLauncherGlobalSerialFilterIntegrationTest.java |  52 +++
 ...alSerialFilterPropertyBlankIntegrationTest.java |  54 
 ...alSerialFilterPropertyEmptyIntegrationTest.java |  54 
 ...atorLauncherJmxSerialFilterIntegrationTest.java |  84 +
 ...mxSerialFilterPropertyBlankIntegrationTest.java |  88 ++
 ...mxSerialFilterPropertyEmptyIntegrationTest.java |  88 ++
 ...xSerialFilterPropertyExistsIntegrationTest.java |  90 ++
 .../filter/LocatorLauncherWithJmxManager.java  |  82 +
 .../filter/SerialFilterAssertions.java |  58 
 ...cherGlobalSerialFilterBlankIntegrationTest.java |  55 
 ...herGlobalSerialFilterExistsIntegrationTest.java |  55 
 ...rLauncherGlobalSerialFilterIntegrationTest.java |  51 +++
 ...alSerialFilterPropertyBlankIntegrationTest.java |  53 
 ...alSerialFilterPropertyEmptyIntegrationTest.java |  53 
 ...rverLauncherJmxSerialFilterIntegrationTest.java |  87 +
 ...mxSerialFilterPropertyBlankIntegrationTest.java |  88 ++
 ...mxSerialFilterPropertyEmptyIntegrationTest.java |  88 ++
 ...xSerialFilterPropertyExistsIntegrationTest.java |  90 ++
 .../filter/ServerLauncherWithJmxManager.java   |  77 +
 ...erConfiguresJmxSerialFilterIntegrationTest.java | 204 
 .../internal/ManagementAgentIntegrationTest.java   |   6 +-
 ...erConfiguresJmxSerialFilterIntegrationTest.java | 201 
 .../apache/geode/distributed/AbstractLauncher.java |   2 +-
 .../apache/geode/distributed/LocatorLauncher.java  |  20 ++
 .../apache/geode/distributed/ServerLauncher.java   | 

[geode] branch develop updated (b5d6917 -> 86b5f7b)

2022-03-29 Thread jbarrett
This is an automated email from the ASF dual-hosted git repository.

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


from b5d6917  GEODE-10182: Remove no longer needed null check. (#7504)
 add 86b5f7b  GEODE-10127: Adds test to preserve correct marshalling 
behavior. (#7461)

No new revisions were added by this update.

Summary of changes:
 .../remote/DistributionLocatorIdJUnitTest.java |  76 --
 .../admin/remote/DistributionLocatorIdTest.java| 141 ++
 .../wan/internal/client/locator/LocatorHelper.java |  22 ++-
 .../internal/client/locator/LocatorHelperTest.java | 163 +
 4 files changed, 314 insertions(+), 88 deletions(-)
 delete mode 100644 
geode-core/src/test/java/org/apache/geode/internal/admin/remote/DistributionLocatorIdJUnitTest.java
 create mode 100644 
geode-core/src/test/java/org/apache/geode/internal/admin/remote/DistributionLocatorIdTest.java
 create mode 100644 
geode-wan/src/test/java/org/apache/geode/cache/wan/internal/client/locator/LocatorHelperTest.java


[geode] branch develop updated (fef6b16 -> b5d6917)

2022-03-29 Thread eshu11
This is an automated email from the ASF dual-hosted git repository.

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


from fef6b16  GEODE-10119: Handle SSLHandshakeException on JDK17 (#7475)
 add b5d6917  GEODE-10182: Remove no longer needed null check. (#7504)

No new revisions were added by this update.

Summary of changes:
 .../src/main/java/org/apache/geode/internal/cache/FilterProfile.java  | 4 
 1 file changed, 4 deletions(-)


[geode] branch develop updated (d8e010d -> fef6b16)

2022-03-29 Thread dhemery
This is an automated email from the ASF dual-hosted git repository.

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


from d8e010d  GEODE-10166: Move tests to correct directory (#7491)
 add fef6b16  GEODE-10119: Handle SSLHandshakeException on JDK17 (#7475)

No new revisions were added by this update.

Summary of changes:
 .../apache/geode/cache/client/internal/ConnectionFactoryImpl.java  | 3 ++-
 .../src/main/java/org/apache/geode/internal/net/SocketCreator.java | 7 +--
 2 files changed, 7 insertions(+), 3 deletions(-)


[geode] branch develop updated (41844a5 -> d8e010d)

2022-03-29 Thread donalevans
This is an automated email from the ASF dual-hosted git repository.

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


from 41844a5  GEODE-10179: Bump jackson-databind to 2.13.2.1 (#7500)
 add d8e010d  GEODE-10166: Move tests to correct directory (#7491)

No new revisions were added by this update.

Summary of changes:
 .../internal/cli/commands/RebalanceCommandAcceptanceTest.java| 9 +
 .../NestedQueryClassCastExceptionFailureDistributedTest.java | 5 +++--
 .../cache/InternalCacheForClientAccessDistributedTest.java   | 2 +-
 .../test/dunit/internal/sanctioned-geode-dunit-serializables.txt | 2 --
 4 files changed, 9 insertions(+), 9 deletions(-)
 rename 
geode-dunit/src/main/java/org/apache/geode/management/internal/cli/commands/RebalanceCommandDistributedTest.java
 => 
geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/RebalanceCommandAcceptanceTest.java
 (96%)
 rename 
geode-dunit/src/main/java/org/apache/geode/management/internal/cli/commands/NestedQueryClassCastExceptionFailureDUnitTest.java
 => 
geode-core/src/distributedTest/java/org/apache/geode/cache/query/dunit/NestedQueryClassCastExceptionFailureDistributedTest.java
 (95%)
 rename 
geode-dunit/src/main/java/org/apache/geode/internal/cache/InternalCacheForClientAccessDUnitTest.java
 => 
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/InternalCacheForClientAccessDistributedTest.java
 (98%)


[geode] branch support/1.13 updated (26f5254 -> ca41979)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

onichols pushed a change to branch support/1.13
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 26f5254  GEODE-10164: Revert wording change in rebalance result (#7490)
 add ca41979  GEODE-9298: remove concourse deprecation warnings

No new revisions were added by this update.

Summary of changes:
 ci/pipelines/pull-request/jinja.template.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


[geode] branch develop updated (5272cbb -> 41844a5)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

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


from 5272cbb  user guide: typo repair in troubleshooting section (#7508)
 add 41844a5  GEODE-10179: Bump jackson-databind to 2.13.2.1 (#7500)

No new revisions were added by this update.

Summary of changes:
 boms/geode-all-bom/src/test/resources/expected-pom.xml   | 7 ++-
 .../org/apache/geode/gradle/plugins/DependencyConstraints.groovy | 9 +
 .../src/integrationTest/resources/assembly_content.txt   | 2 +-
 .../src/integrationTest/resources/gfsh_dependency_classpath.txt  | 2 +-
 .../src/integrationTest/resources/dependency_classpath.txt   | 2 +-
 5 files changed, 18 insertions(+), 4 deletions(-)


[geode] branch support/1.14 updated (483e54c -> 3f62606)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

onichols pushed a change to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 483e54c  GEODE-9298: remove concourse deprecation warnings
 add 3f62606  GEODE-10179: Bump jackson-databind to 2.12.6.1 (#7501)

No new revisions were added by this update.

Summary of changes:
 boms/geode-all-bom/src/test/resources/expected-pom.xml | 10 +-
 .../apache/geode/gradle/plugins/DependencyConstraints.groovy   |  7 +--
 .../src/integrationTest/resources/assembly_content.txt |  6 +++---
 .../src/integrationTest/resources/dependency_classpath.txt |  6 +++---
 4 files changed, 16 insertions(+), 13 deletions(-)


[geode] branch support/1.12 updated (fdd5c2c -> 36d9260)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

onichols pushed a change to branch support/1.12
in repository https://gitbox.apache.org/repos/asf/geode.git.


from fdd5c2c  GEODE-10164: Revert wording change in rebalance result (#7490)
 add 36d9260  GEODE-10179: Bump jackson-databind from 2.10.5.1 to 2.12.6.1 
(#7506)

No new revisions were added by this update.

Summary of changes:
 .../src/test/resources/expected-pom.xml| 10 +-
 .../gradle/plugins/DependencyConstraints.groovy|  8 
 .../integrationTest/resources/assembly_content.txt |  6 +++---
 .../resources/dependency_classpath.txt |  6 +++---
 .../DistributedSystemMBeanIntegrationTest.java | 22 +++---
 5 files changed, 30 insertions(+), 22 deletions(-)


[geode] branch support/1.13 updated (ca41979 -> 7e18d73)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

onichols pushed a change to branch support/1.13
in repository https://gitbox.apache.org/repos/asf/geode.git.


from ca41979  GEODE-9298: remove concourse deprecation warnings
 add 7e18d73  [1.13] GEODE-10179: Bump jackson-databind from 2.10.5.1 to 
2.12.6.1 (#7503)

No new revisions were added by this update.

Summary of changes:
 .../src/test/resources/expected-pom.xml| 10 +-
 .../gradle/plugins/DependencyConstraints.groovy|  8 
 .../integrationTest/resources/assembly_content.txt |  6 +++---
 .../resources/dependency_classpath.txt |  6 +++---
 .../DistributedSystemMBeanIntegrationTest.java | 22 +++---
 5 files changed, 30 insertions(+), 22 deletions(-)


[geode] branch develop updated (dfedcbd -> 5272cbb)

2022-03-29 Thread dbarnes
This is an automated email from the ASF dual-hosted git repository.

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


from dfedcbd  GEODE-10140: Opens/exports for QueryResultFormatter tests 
(#7481)
 add 5272cbb  user guide: typo repair in troubleshooting section (#7508)

No new revisions were added by this update.

Summary of changes:
 .../managing/troubleshooting/log_messages_and_solutions.html.md.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


[geode] branch support/1.14 updated (6bf622d -> 483e54c)

2022-03-29 Thread onichols
This is an automated email from the ASF dual-hosted git repository.

onichols pushed a change to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git.


from 6bf622d  GEODE-10164: Revert wording change in rebalance result (#7490)
 add 483e54c  GEODE-9298: remove concourse deprecation warnings

No new revisions were added by this update.

Summary of changes:
 ci/pipelines/pull-request/jinja.template.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)