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 57cc931 GEODE-7710: Add getRedundancyZone to DistributionManager (#4809) add 48ea49c GEODE-7869: Cleanup warnings in geode-http-service add 5abf502 GEODE-7869: Cleanup warnings in geode-common add f098829 GEODE-7869: Cleanup warnings in geode-concurrency-test add 4fb85db GEODE-7869: Cleanup warnings in geode-connectors add fa9bf79 GEODE-7869: Cleanup warnings in geode-management add ebdae67 GEODE-7869: Cleanup warnings in geode-rebalancer add 0f6f28b GEODE-7869: Cleanup warnings in geode-serialization add b04a072 GEODE-7869: Cleanup warnings in geode-unsafe add 9b269de GEODE-7869: Cleanup warnings in geode-web add 9d1e682 GEODE-7869: Cleanup warnings in geode-web-api add 95e88d9 GEODE-7869: Cleanup warnings in geode-web-management add ba258cc GEODE-7869: Cleanup warnings in static-analysis add 8e4601e GEODE-7869: Cleanup warnings in geode-experimental-driver add feab561 GEODE-7869: Cleanup warnings in geode-assembly add d7a2bae GEODE-7869: Cleanup warnings in geode-pulse add 3bd72e2 GEODE-7869: Cleanup warnings in geode-protobuf add 644c52f GEODE-7869: Cleanup warnings in geode-protobuf-messages add 23701b2 GEODE-7869: Cleanup warnings in geode-redis add 028777b GEODE-7869: Cleanup warnings in geode-tcp-server No new revisions were added by this update. Summary of changes: .../src/test/resources/expected-pom.xml | 6 + .../gradle/plugins/DependencyConstraints.groovy | 1 + geode-assembly/build.gradle | 1 + .../cli/commands/ImportClusterConfigTest.java | 2 +- .../cli/commands/RunOutOfMemoryFunction.java | 6 +- ...StatusClusterConfigServiceCommandDUnitTest.java | 7 +- .../cli/commands/StopLocatorCommandDUnitTest.java | 38 +- ...ClusterManagementLocatorReconnectDunitTest.java | 9 +- .../rest/DeploymentManagementRedployDUnitTest.java | 23 +- .../internal/rest/RegionManagementDunitTest.java | 37 +- .../internal/rest/ServerRestartTest.java | 7 +- .../rest/internal/web/RestFunctionTemplate.java | 4 +- .../controllers/RestAPIsAndInterOpsDUnitTest.java | 11 +- .../web/controllers/RestAPIsWithSSLDUnitTest.java | 34 +- .../internal/web/RestInterfaceIntegrationTest.java | 32 +- .../rest/internal/web/RestRegionAPIDUnitTest.java | 22 +- .../internal/web/RestSecurityIntegrationTest.java | 54 +- .../web/RestSecurityPostProcessorTest.java | 20 +- .../tools/pulse/PulseSecurityWithSSLTest.java | 4 +- .../integrationTest/resources/expected_jars.txt | 1 + geode-common/build.gradle | 1 + geode-concurrency-test/build.gradle | 1 + geode-connectors/build.gradle | 1 + .../jdbc/JdbcAsyncWriterIntegrationTest.java | 12 +- .../connectors/jdbc/JdbcWriterIntegrationTest.java | 16 +- .../jdbc/internal/TestConfigService.java | 5 +- .../cli/CreateDataSourceCommandDUnitTest.java | 21 +- ...reateMappingCommandForProxyRegionDUnitTest.java | 23 +- .../cli/DeregisterDriverCommandDUnitTest.java | 14 +- .../cli/DescribeDataSourceCommandDUnitTest.java | 15 +- .../cli/DescribeMappingCommandDUnitTest.java | 40 +- .../cli/DestroyDataSourceCommandDUnitTest.java | 10 +- .../cli/ListDataSourceCommandDUnitTest.java | 17 +- .../internal/cli/ListDriversCommandDUnitTest.java | 9 +- .../internal/cli/ListMappingCommandDUnitTest.java | 13 +- .../cli/RegisterDriverCommandDUnitTest.java | 13 +- .../cli/JDBCConnectorFunctionsSecurityTest.java | 20 +- .../geode/connectors/jdbc/JdbcAsyncWriter.java | 37 +- .../apache/geode/connectors/jdbc/JdbcLoader.java | 1 + .../jdbc/internal/cli/CreateMappingCommand.java | 9 +- .../jdbc/internal/cli/CreateMappingFunction.java | 11 +- .../internal/cli/DeregisterDriverFunction.java | 2 +- .../jdbc/internal/cli/ListDriversCommand.java | 5 +- .../jdbc/internal/cli/RegisterDriverFunction.java | 2 +- .../geode/connectors/jdbc/JdbcLoaderTest.java | 9 +- .../geode/connectors/jdbc/JdbcWriterTest.java | 12 +- .../jdbc/internal/JdbcConnectorServiceTest.java | 9 +- .../internal/cli/CreateDataSourceCommandTest.java | 7 +- .../internal/cli/CreateMappingFunctionTest.java | 27 +- ...CreateMappingPreconditionCheckFunctionTest.java | 93 ++-- .../internal/cli/DeregisterDriverCommandTest.java | 15 +- .../internal/cli/DeregisterDriverFunctionTest.java | 3 +- .../internal/cli/DestroyDataSourceCommandTest.java | 10 +- .../cli/DestroyMappingCommandFunctionTest.java | 24 +- .../cli/FunctionContextArgumentProviderTest.java | 18 +- .../jdbc/internal/cli/ListDriversCommandTest.java | 1 + .../jdbc/internal/cli/ListDriversFunctionTest.java | 1 + .../jdbc/internal/cli/ListMappingCommandTest.java | 22 +- .../internal/cli/RegisterDriverCommandTest.java | 6 +- .../internal/cli/RegisterDriverFunctionTest.java | 7 +- .../jdbc/internal/xml/ElementTypeTest.java | 8 +- .../xml/JdbcConnectorServiceXmlParserTest.java | 4 +- .../query/partitioned/PRColocatedEquiJoinTest.java | 42 +- .../internal/jta/TransactionImplJUnitTest.java | 6 +- .../cache/partitioned/IndexCreationMsg.java | 2 +- ...currentParallelGatewaySenderEventProcessor.java | 2 +- ...oncurrentSerialGatewaySenderEventProcessor.java | 2 +- .../geode/test/junit/rules/MemberStarterRule.java | 11 +- geode-experimental-driver/build.gradle | 1 + .../experimental/driver/AuthorizationTest.java | 3 +- .../experimental/driver/IntegrationTestBase.java | 6 +- .../driver/PostProcessingIntegrationTest.java | 5 +- .../geode/experimental/driver/FunctionService.java | 2 +- .../geode/experimental/driver/ProtobufDriver.java | 10 +- .../experimental/driver/ProtobufFunction.java | 11 +- .../driver/ProtobufFunctionService.java | 4 +- .../experimental/driver/ProtobufQueryService.java | 18 +- .../geode/experimental/driver/ProtobufRegion.java | 12 +- .../geode/experimental/driver/QueryService.java | 2 +- .../geode/experimental/driver/ValueEncoder.java | 10 +- geode-http-service/build.gradle | 1 + geode-management/build.gradle | 1 + .../api/ClusterManagementListResult.java | 15 +- ...tTemplateClusterManagementServiceTransport.java | 4 +- .../internal/api/BaseManagementServiceBuilder.java | 22 +- .../api/ClusterManagementGetResultTest.java | 1 + .../api/ClusterManagementListResultTest.java | 4 +- .../api/ClusterManagementOperationResultTest.java | 6 +- .../geode/management/api/EntityInfoTest.java | 1 + ...plateClusterManagementServiceTransportTest.java | 18 +- .../ClusterManagementServiceBuilderTest.java | 3 +- .../ClientClusterManagementServiceTest.java | 4 + geode-protobuf-messages/build.gradle | 1 + geode-protobuf/build.gradle | 1 + .../v1/ValueSerializerIntegrationTest.java | 9 +- .../protobuf/security/SecureCacheImpl.java | 35 +- .../security/SecureFunctionServiceImpl.java | 27 +- .../statistics/ProtobufClientStatistics.java | 42 +- .../protocol/protobuf/v1/ProtobufOpsProcessor.java | 26 +- .../OqlQueryRequestOperationHandler.java | 4 +- .../protocol/protobuf/v1/state/RequireVersion.java | 7 +- .../protobuf/security/SecureCacheImplTest.java | 41 +- .../security/SecureFunctionServiceImplTest.java | 37 +- .../ClearRequestOperationHandlerJUnitTest.java | 21 +- ...ionOnGroupRequestOperationHandlerJUnitTest.java | 45 +- ...onOnMemberRequestOperationHandlerJUnitTest.java | 58 +- ...onOnRegionRequestOperationHandlerJUnitTest.java | 43 +- .../GetAllRequestOperationHandlerJUnitTest.java | 79 +-- ...egionNamesRequestOperationHandlerJUnitTest.java | 52 +- .../GetRequestOperationHandlerJUnitTest.java | 65 +-- .../GetServerOperationHandlerJUnitTest.java | 54 +- .../GetSizeRequestOperationHandlerJUnitTest.java | 19 +- .../KeySetOperationHandlerJUnitTest.java | 26 +- .../v1/operations/OperationHandlerJUnitTest.java | 4 +- ...ueryRequestOperationHandlerIntegrationTest.java | 51 +- .../OqlQueryRequestOperationHandlerJUnitTest.java | 11 +- .../PutAllRequestOperationHandlerJUnitTest.java | 74 +-- .../PutRequestOperationHandlerJUnitTest.java | 28 +- .../RemoveRequestOperationHandlerJUnitTest.java | 57 +- .../codec/JsonPdxConverterJUnitTest.java | 16 +- geode-pulse/build.gradle | 2 + geode-pulse/geode-pulse-test/build.gradle | 1 + .../controllers/PulseControllerJUnitTest.java | 183 +++---- .../internal/controllers/PulseController.java | 95 ++-- .../geode/tools/pulse/internal/data/Cluster.java | 602 ++++++++++----------- .../tools/pulse/internal/data/DataBrowser.java | 4 +- .../tools/pulse/internal/data/JMXDataUpdater.java | 304 +++++------ .../internal/security/DefaultSecurityConfig.java | 7 +- .../service/ClusterDiskThroughputService.java | 4 +- .../internal/service/ClusterGCPausesService.java | 2 +- .../service/ClusterKeyStatisticsService.java | 6 +- .../internal/service/ClusterMemberService.java | 14 +- .../service/ClusterMembersRGraphService.java | 148 +++-- .../service/ClusterMemoryUsageService.java | 2 +- .../internal/service/ClusterRegionService.java | 38 +- .../internal/service/ClusterRegionsService.java | 47 +- .../service/ClusterSelectedRegionService.java | 31 +- .../ClusterSelectedRegionsMemberService.java | 31 +- .../internal/service/ClusterWANInfoService.java | 4 +- .../service/MemberAsynchEventQueuesService.java | 2 +- .../internal/service/MemberClientsService.java | 16 +- .../service/MemberDiskThroughputService.java | 6 +- .../internal/service/MemberGCPausesService.java | 4 +- .../internal/service/MemberGatewayHubService.java | 11 +- .../internal/service/MemberHeapUsageService.java | 4 +- .../service/MemberKeyStatisticsService.java | 8 +- .../internal/service/MemberRegionsService.java | 28 +- .../pulse/internal/service/MembersListService.java | 2 +- .../internal/service/QueryStatisticsService.java | 90 +-- .../internal/service/SystemAlertsService.java | 14 +- .../ClusterSelectedRegionsMemberServiceTest.java | 55 +- .../geode/tools/pulse/tests/ui/PulseBase.java | 117 +--- geode-rebalancer/build.gradle | 1 + geode-redis/build.gradle | 1 + .../org/apache/geode/redis/RedisDistDUnitTest.java | 37 +- .../redis/ConcurrentStartIntegrationTest.java | 31 +- .../org/apache/geode/redis/GeodeRedisServer.java | 120 ++-- .../java/org/apache/geode/redis/internal/Pair.java | 30 - .../apache/geode/redis/internal/PubSubImpl.java | 11 +- .../geode/redis/internal/RegionProvider.java | 116 ++-- .../redis/internal/executor/RenameExecutor.java | 7 +- .../internal/executor/list/LIndexExecutor.java | 6 +- .../redis/internal/executor/list/LLenExecutor.java | 8 +- .../internal/executor/list/LRangeExecutor.java | 13 +- .../redis/internal/executor/list/LRemExecutor.java | 11 +- .../redis/internal/executor/list/LSetExecutor.java | 13 +- .../internal/executor/list/LTrimExecutor.java | 15 +- .../redis/internal/executor/list/ListExecutor.java | 20 +- .../redis/internal/executor/list/PopExecutor.java | 8 +- .../redis/internal/executor/list/PushExecutor.java | 4 +- .../internal/executor/list/PushXExecutor.java | 4 +- .../executor/sortedset/GeoSortedSetExecutor.java | 1 + .../executor/sortedset/ZRangeByLexExecutor.java | 9 +- .../executor/sortedset/ZRangeByScoreExecutor.java | 14 +- .../internal/ExecutionHandlerContextJUnitTest.java | 4 +- .../string/StringSetExecutorJUnitTest.java | 4 +- geode-serialization/build.gradle | 1 + .../serialization/BufferDataOutputStream.java | 333 ++++++------ .../internal/serialization/ByteArrayDataInput.java | 134 ++--- .../internal/serialization/DSFIDSerializer.java | 2 +- .../serialization/StaticSerialization.java | 14 +- .../internal/DSFIDSerializerImpl.java | 22 +- geode-tcp-server/build.gradle | 1 + .../TcpServerProductVersionDUnitTest.java | 24 +- .../internal/tcpserver/InfoRequest.java | 1 + .../distributed/internal/tcpserver/TcpClient.java | 3 +- .../internal/tcpserver/TcpServerJUnitTest.java | 47 +- geode-unsafe/build.gradle | 1 + geode-web-api/build.gradle | 1 + .../web/controllers/AddFreeItemToOrders.java | 45 +- .../web/controllers/EchoArgumentFunction.java | 4 +- .../controllers/GetOrderDescriptionFunction.java | 23 +- .../geode/rest/internal/web/controllers/Order.java | 39 +- .../web/controllers/RestAccessControllerTest.java | 21 +- .../web/controllers/AbstractBaseController.java | 54 +- .../web/controllers/CommonCrudController.java | 19 +- .../web/controllers/FunctionAccessController.java | 122 +++-- .../web/controllers/PdxBasedCrudController.java | 12 +- .../web/controllers/QueryAccessController.java | 9 +- .../web/controllers/support/RegionData.java | 13 +- .../support/RestServersResultCollector.java | 6 +- .../geode/rest/internal/web/util/ArrayUtils.java | 11 +- .../geode/rest/internal/web/util/JSONUtils.java | 39 +- geode-web-management/build.gradle | 1 + .../rest/security/RestSecurityConfiguration.java | 2 +- geode-web/build.gradle | 1 + .../cli/commands/ConnectCommandWithSSLTest.java | 109 ++-- ...xecuteFunctionCommandWithSecurityDUnitTest.java | 6 +- .../ExportLogsStatsOverHttpDistributedTest.java | 8 +- ...rverSerializableObjectHttpMessageConverter.java | 2 +- static-analysis/build.gradle | 1 + static-analysis/pmd-rules/build.gradle | 1 + 212 files changed, 2726 insertions(+), 2831 deletions(-) delete mode 100644 geode-redis/src/main/java/org/apache/geode/redis/internal/Pair.java