This is an automated email from the ASF dual-hosted git repository. bschuchardt pushed a change to branch feature/GEODE-7090 in repository https://gitbox.apache.org/repos/asf/geode.git.
discard 32bc462 addressing suspect string in ClusterConfigLocatorRestartDUnitTest discard 9d2b6dd fixing Jetty classpath to include geode-serialization jar discard ab7b074 rebased on origin & fixed deserialization issue discard 11601a0 splitting ObjectSerializer into a serializer and a deserializer discard 39d0371 incorporating review comments and fixing tomcat test classpath discard 4cd3f3b GEODE-7090 Remove dependency on DataSerializer from membership classes add 7cde6cc2 GEODE-7118: Cleanup RestartOfMemberDistributedTest add dafa5f0 Add standalone LogWriterLogger to geode-dunit add c79cd1d GEODE-7114: fix flaky tests by using GeodeAwaitility. (#3979) add 7f2b986 GEODE-7039: Improve read performance during server recovery (#3955) add 78cef46 GEODE-7136: remove "groups" in management configuration (#3997) add 245f285 GEODE-6916: Add Aggregate Functions Tests (#3953) add 57f1cc7 GEODE-7158: Adding singleton cleanup before and after tests add 5ef067d GEODE-7090 Remove dependency on DataSerializer from membership classes add 8020751 incorporating review comments and fixing tomcat test classpath add 480f9ba splitting ObjectSerializer into a serializer and a deserializer add b284361 rebased on origin & fixed deserialization issue add 9b3584b fixing Jetty classpath to include geode-serialization jar add 2bb5a68 addressing suspect string in ClusterConfigLocatorRestartDUnitTest 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 (32bc462) \ N -- N -- N refs/heads/feature/GEODE-7090 (2bb5a68) 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: .../rest/ListRegionManagementDunitTest.java | 24 +- .../geode/cache/query/dunit/GroupByDUnitImpl.java | 334 ---- .../dunit/GroupByPartitionedQueryDUnitTest.java | 93 - .../cache/query/dunit/GroupByQueryDUnitTest.java | 189 -- .../dunit/PdxGroupByPartitionedQueryDUnitTest.java | 92 - .../AggregateFunctionsQueryDUnitTest.java | 1857 ++++++++++++++++++++ ...lusterDistributionManagerForAdminDUnitTest.java | 6 +- .../internal/RestartOfMemberDistributedTest.java | 81 +- ...ntPartitionedRegionWithRedundancyDUnitTest.java | 318 ++++ .../jta/dunit/TransactionTimeOutDUnitTest.java | 188 +- .../cli/commands/CreateIndexCommandDUnitTest.java | 13 +- .../cache/query/functional/CountStarJUnitTest.java | 674 ------- .../functional/GroupByPartitionedJUnitTest.java | 43 - .../functional/GroupByReplicatedJUnitTest.java | 32 - .../functional/PdxGroupByPartitionedJUnitTest.java | 38 - .../functional/PdxGroupByReplicatedJUnitTest.java | 35 - .../AggregateFunctionQueryBaseIntegrationTest.java | 86 + .../internal/aggregate/AvgIntegrationTest.java | 337 ++++ .../internal/aggregate/CountIntegrationTest.java | 340 ++++ .../internal/aggregate/MaxIntegrationTest.java | 334 ++++ .../internal/aggregate/MinIntegrationTest.java | 334 ++++ .../internal/aggregate/SumIntegrationTest.java | 328 ++++ .../ConfigurationPersistenceService.java | 3 +- .../apache/geode/internal/cache/BucketAdvisor.java | 32 +- .../api/LocatorClusterManagementService.java | 79 +- .../internal/cli/commands/CreateIndexCommand.java | 43 +- ...ator.java => CommonConfigurationValidator.java} | 8 +- .../configuration/validators/MemberValidator.java | 15 +- .../sanctioned-geode-management-serializables.txt | 2 +- .../tier/sockets/CacheClientNotifierTest.java | 20 + .../api/LocatorClusterManagementServiceTest.java | 66 +- .../cli/commands/CreateIndexCommandTest.java | 99 +- ....java => CommonConfigurationValidatorTest.java} | 17 +- .../validators/MemberValidatorTest.java | 48 +- .../apache/geode/test/dunit/LogWriterUtils.java | 51 +- .../geode/test/dunit}/log4j/LogWriterLogger.java | 516 +++++- .../geode/test/dunit/rules/ClusterStartupRule.java | 1 + .../apache/geode/test/junit/rules/VMProvider.java | 15 +- .../cache/query/functional/GroupByTestImpl.java | 1031 ----------- .../query/functional/GroupByTestInterface.java | 58 - .../cache/query/functional/PdxGroupByTestImpl.java | 1025 ----------- .../management/api/ClusterManagementService.java | 4 +- .../configuration/AbstractConfiguration.java | 59 +- .../management/configuration/GatewayReceiver.java | 6 +- .../apache/geode/management/configuration/Pdx.java | 2 +- .../configuration/AbstractConfigurationTest.java | 49 +- .../ConfigurationJsonMappingTest.java | 33 +- .../controllers/MemberManagementController.java | 3 + 48 files changed, 4863 insertions(+), 4198 deletions(-) delete mode 100644 geode-core/src/distributedTest/java/org/apache/geode/cache/query/dunit/GroupByDUnitImpl.java delete mode 100644 geode-core/src/distributedTest/java/org/apache/geode/cache/query/dunit/GroupByPartitionedQueryDUnitTest.java delete mode 100644 geode-core/src/distributedTest/java/org/apache/geode/cache/query/dunit/GroupByQueryDUnitTest.java delete mode 100644 geode-core/src/distributedTest/java/org/apache/geode/cache/query/dunit/PdxGroupByPartitionedQueryDUnitTest.java create mode 100644 geode-core/src/distributedTest/java/org/apache/geode/cache/query/internal/aggregate/AggregateFunctionsQueryDUnitTest.java create mode 100644 geode-core/src/distributedTest/java/org/apache/geode/internal/cache/persistence/PersistentPartitionedRegionWithRedundancyDUnitTest.java delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/functional/CountStarJUnitTest.java delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/functional/GroupByPartitionedJUnitTest.java delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/functional/GroupByReplicatedJUnitTest.java delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/functional/PdxGroupByPartitionedJUnitTest.java delete mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/functional/PdxGroupByReplicatedJUnitTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/AggregateFunctionQueryBaseIntegrationTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/AvgIntegrationTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/CountIntegrationTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/MaxIntegrationTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/MinIntegrationTest.java create mode 100644 geode-core/src/integrationTest/java/org/apache/geode/cache/query/internal/aggregate/SumIntegrationTest.java rename geode-core/src/main/java/org/apache/geode/management/internal/configuration/validators/{CacheElementValidator.java => CommonConfigurationValidator.java} (88%) rename geode-core/src/test/java/org/apache/geode/management/internal/configuration/validators/{CacheElementValidatorTest.java => CommonConfigurationValidatorTest.java} (82%) copy {geode-core/src/main/java/org/apache/geode/internal/logging => geode-dunit/src/main/java/org/apache/geode/test/dunit}/log4j/LogWriterLogger.java (80%) delete mode 100644 geode-junit/src/main/java/org/apache/geode/cache/query/functional/GroupByTestImpl.java delete mode 100644 geode-junit/src/main/java/org/apache/geode/cache/query/functional/GroupByTestInterface.java delete mode 100644 geode-junit/src/main/java/org/apache/geode/cache/query/functional/PdxGroupByTestImpl.java