This is an automated email from the ASF dual-hosted git repository. zhouxj pushed a change to branch feature/GEODE-3967 in repository https://gitbox.apache.org/repos/asf/geode.git.
omit eb48af8 GEODE-3967: There're following 5 problems fixed here: 1) When ConcurrentCacheModificationException happened, GatewaySenderEventImpl should save the status and notify gatewaysender if it hold primary queue, because other member might have put the event into the secondary queue. 2) In AbstractUpdateOperation's doPutOrCreate's 3 tries of basicUpdate, the 3rd try should allow both create and update. 3) Let event with CME not to dispatch. The old logic does not allow CME eve [...] add 906ed2a GEODE-3948 Improve CQ performance under flaky network conditions add 6c331d1 GEODE-4499: Test adding a compressor to an existing persistent region. (#1384) add 69443da GEODE-4598: Make LRUListWithAsyncSorting the default eviction list (#1387) add ff1b8c7 GEODE-4389: Ensure that region template creation does not require callback classes on the locator (#1390) add e3b0666 Fix capture_call_stacks.sh (#1393) new d6332e0 GEODE-3967: There're following 6 problems fixed here: 1) When ConcurrentCacheModificationException happened, GatewaySenderEventImpl should save the status and notify gatewaysender if it hold primary queue, because other member might have put the event into the secondary queue. 2) In AbstractUpdateOperation's doPutOrCreate's 3 tries of basicUpdate, the 3rd try should allow both create and update. 3) Let event with CME not to dispatch. The old logic does not allow CME eve [...] 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 (eb48af8) \ N -- N -- N refs/heads/feature/GEODE-3967 (d6332e0) 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: ci/docker/Dockerfile | 4 +- ci/scripts/capture-call-stacks.sh | 2 +- .../java/org/apache/geode/cache/client/Pool.java | 10 + .../org/apache/geode/cache/client/PoolFactory.java | 20 ++ .../geode/cache/client/internal/AbstractOp.java | 2 +- .../cache/client/internal/AuthenticateUserOp.java | 2 +- .../client/internal/ClientSideHandshakeImpl.java | 32 ++- .../geode/cache/client/internal/PoolImpl.java | 10 + .../apache/geode/cache/client/internal/PutOp.java | 2 +- .../internal/pooling/ConnectionManagerImpl.java | 1 - .../java/org/apache/geode/internal/Version.java | 8 +- .../geode/internal/cache/PoolFactoryImpl.java | 36 ++- .../cache/eviction/EvictionListBuilder.java | 10 +- .../cache/eviction/LRUListWithAsyncSorting.java | 1 - .../apache/geode/internal/cache/properties.html | 4 +- .../internal/cache/tier/ServerSideHandshake.java | 5 +- .../internal/cache/tier/sockets/AcceptorImpl.java | 5 +- .../internal/cache/tier/sockets/BaseCommand.java | 2 +- .../cache/tier/sockets/CacheClientNotifier.java | 23 +- .../cache/tier/sockets/CacheClientUpdater.java | 29 ++- .../cache/tier/sockets/CommandInitializer.java | 5 + .../geode/internal/cache/tier/sockets/Message.java | 130 ++++++----- .../tier/sockets/OriginalServerConnection.java | 4 +- .../tier/sockets/ServerHandshakeProcessor.java | 2 + .../cache/tier/sockets/ServerQueueStatus.java | 32 ++- .../tier/sockets/ServerSideHandshakeImpl.java | 5 +- .../wan/AbstractGatewaySenderEventProcessor.java | 2 +- .../geode/internal/lang/SystemPropertyHelper.java | 41 ++-- .../internal/cli/commands/CreateRegionCommand.java | 56 +++-- .../functions/FetchRegionAttributesFunction.java | 61 ++++- .../cli/functions/RegionAttributesWrapper.java | 108 +++++++++ .../sanctioned-geode-core-serializables.txt | 1 + .../AutoConnectionSourceImplJUnitTest.java | 5 + .../client/internal/QueueManagerJUnitTest.java | 43 +++- .../cache/eviction/EvictionListBuilderTest.java | 80 +++++++ .../tier/sockets/ClientServerMiscDUnitTest.java | 55 +++++ .../cache/tier/sockets/MessageJUnitTest.java | 58 +++++ .../cache/tier/sockets/ServerConnectionTest.java | 2 +- ...UnitTest.java => SystemPropertyHelperTest.java} | 43 +++- .../cli/commands/AlterCompressorDUnitTest.java | 249 +++++++++++++++++++++ .../cli/commands/CreateRegionCommandDUnitTest.java | 155 +++++++++++++ .../{TestCompressor.java => TestCompressor1.java} | 15 +- .../tier/sockets/ClientServerMiscBCDUnitTest.java | 10 + .../client/internal/GatewaySenderBatchOp.java | 3 +- .../RemoteParallelGatewaySenderEventProcessor.java | 35 +-- .../RemoteSerialGatewaySenderEventProcessor.java | 19 ++ 46 files changed, 1208 insertions(+), 219 deletions(-) create mode 100644 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/RegionAttributesWrapper.java create mode 100644 geode-core/src/test/java/org/apache/geode/internal/cache/eviction/EvictionListBuilderTest.java rename geode-core/src/test/java/org/apache/geode/internal/lang/{SystemPropertyHelperJUnitTest.java => SystemPropertyHelperTest.java} (65%) create mode 100644 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/AlterCompressorDUnitTest.java copy geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/{TestCompressor.java => TestCompressor1.java} (61%) -- To stop receiving notification emails like this one, please contact zho...@apache.org.