[geode] branch develop updated (02a5cb7 -> 9377d81)
This is an automated email from the ASF dual-hosted git repository. dschneider pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 02a5cb7 GEODE-9649: using spotless rule to enforce dependency version constraints add 9377d81 GEODE-9607: prevent pubsub from running server out of memory (#6880) No new revisions were added by this update. Summary of changes: .../apache/geode/redis/OutOfMemoryDUnitTest.java | 53 +++- .../apache/geode/codeAnalysis/excludedClasses.txt | 2 +- .../codeAnalysis/sanctionedDataSerializables.txt | 4 + .../geode/redis/internal/GeodeRedisServer.java | 3 +- .../geode/redis/internal/GeodeRedisService.java| 4 + .../geode/redis/internal/RegionProvider.java | 23 ++ .../redis/internal/executor/AbstractExecutor.java | 15 + .../internal/executor/pubsub/PublishExecutor.java | 3 +- .../executor/pubsub/SubscribeExecutor.java | 1 + .../apache/geode/redis/internal/pubsub/PubSub.java | 5 +- .../geode/redis/internal/pubsub/PubSubImpl.java| 167 +-- .../geode/redis/internal/pubsub/Publisher.java | 330 + .../redis/internal/pubsub/PubSubImplTest.java | 5 +- .../geode/redis/internal/pubsub/PublisherTest.java | 205 + .../internal/cache/control/HeapMemoryMonitor.java | 4 +- .../serialization/DataSerializableFixedID.java | 3 +- 16 files changed, 656 insertions(+), 171 deletions(-) create mode 100644 geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/pubsub/Publisher.java create mode 100644 geode-apis-compatible-with-redis/src/test/java/org/apache/geode/redis/internal/pubsub/PublisherTest.java
[geode] branch develop updated (5b78326 -> 02a5cb7)
This is an automated email from the ASF dual-hosted git repository. rhoughton pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 5b78326 GEODE-9623: Add Radish COMMAND command (#6907) add 02a5cb7 GEODE-9649: using spotless rule to enforce dependency version constraints No new revisions were added by this update. Summary of changes: .../src/test/resources/expected-pom.xml | 20 +++- .../gradle/plugins/DependencyConstraints.groovy | 3 +++ geode-apis-compatible-with-redis/build.gradle| 4 ++-- .../src/test/resources/expected-pom.xml | 5 - geode-assembly/build.gradle | 2 ++ geode-core/build.gradle | 2 +- geode-core/src/test/resources/expected-pom.xml | 0 geode-gfsh/build.gradle | 2 +- geode-wan/build.gradle | 2 +- gradle/spotless.gradle | 9 - 10 files changed, 33 insertions(+), 16 deletions(-) mode change 100755 => 100644 geode-core/src/test/resources/expected-pom.xml
[geode-native] branch develop updated: GEODE-9609: Fix Warning in SessionState (#869)
This is an automated email from the ASF dual-hosted git repository.
mmartell 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 1bfc74e GEODE-9609: Fix Warning in SessionState (#869)
1bfc74e is described below
commit 1bfc74eea17bba74226ffc6192ede0c459c2f261
Author: Michael Martell
AuthorDate: Wed Sep 29 12:52:19 2021 -0700
GEODE-9609: Fix Warning in SessionState (#869)
---
netcore/netcore-session/NetCoreSessionState.cs | 17 +++--
1 file changed, 3 insertions(+), 14 deletions(-)
diff --git a/netcore/netcore-session/NetCoreSessionState.cs
b/netcore/netcore-session/NetCoreSessionState.cs
index 95b0434..4bf0b88 100644
--- a/netcore/netcore-session/NetCoreSessionState.cs
+++ b/netcore/netcore-session/NetCoreSessionState.cs
@@ -74,7 +74,6 @@ namespace Apache.Geode.Session {
public class GeodeSessionStateCache : GeodeNativeObject, IDistributedCache {
private readonly IGeodeCache _cache;
-private ILogger _logger;
private static Region _region;
private string _logLevel;
private string _logFile;
@@ -276,19 +275,9 @@ namespace Apache.Geode.Session {
}
_connectLock.Wait();
- try {
-using var regionFactory =
_cache.CreateRegionFactory(RegionShortcut.Proxy);
-try {
- _logger?.LogTrace("Create CacheRegion");
- _region = regionFactory.CreateRegion(_regionName);
- _logger?.LogTrace("CacheRegion created");
-} catch (Exception e) {
- _logger?.LogInformation(e, "Create CacheRegion failed... now trying
to get the region");
-}
- } finally {
-// regionFactory?.Dispose();
-_connectLock.Release();
- }
+ using var regionFactory =
_cache.CreateRegionFactory(RegionShortcut.Proxy);
+ _region = regionFactory.CreateRegion(_regionName);
+ _connectLock.Release();
}
protected override void DestroyContainedObject() {
[geode] branch develop updated (28321af -> 5b78326)
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 28321af GEODE-9516: add ZINTERSTORE command (#6861)
add 5b78326 GEODE-9623: Add Radish COMMAND command (#6907)
No new revisions were added by this update.
Summary of changes:
geode-apis-compatible-with-redis/build.gradle | 1 +
.../java/org/apache/geode/NativeRedisTestRule.java | 9 +-
.../executor/server/CommandIntegrationTest.java| 129 +++
.../apache/geode/codeAnalysis/excludedClasses.txt | 2 +
.../geode/redis/internal/RedisCommandType.java | 370 +
...{FlushAllExecutor.java => CommandExecutor.java} | 38 ++-
.../redis/internal/executor/set/SPopExecutor.java | 7 +-
.../parameters/ClusterParameterRequirements.java | 29 +-
.../parameters/EvenParameterRequirements.java | 51 ---
.../parameters/ExactParameterRequirements.java | 34 --
.../parameters/MaximumParameterRequirements.java | 49 ---
.../parameters/MinimumParameterRequirements.java | 35 --
.../parameters/MultipleParameterRequirements.java | 36 --
.../parameters/OddParameterRequirements.java | 50 ---
.../geode/redis/internal/parameters/Parameter.java | 163 +
.../internal/parameters/ParameterRequirements.java | 27 --
.../parameters/SlowlogParameterRequirements.java | 37 ++-
.../parameters/SpopParameterRequirements.java | 35 --
.../UnspecifiedParameterRequirements.java | 27 --
...de-apis-compatible-with-redis-serializables.txt | 1 -
20 files changed, 597 insertions(+), 533 deletions(-)
create mode 100644
geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/server/CommandIntegrationTest.java
copy
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/executor/server/{FlushAllExecutor.java
=> CommandExecutor.java} (58%)
mode change 100755 => 100644
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/EvenParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/ExactParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/MaximumParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/MinimumParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/MultipleParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/OddParameterRequirements.java
create mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/Parameter.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/ParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/SpopParameterRequirements.java
delete mode 100644
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/parameters/UnspecifiedParameterRequirements.java
[geode] 02/02: GEODE-9486: Improve AnalyzeSerializables integration tests (#6878)
This is an automated email from the ASF dual-hosted git repository.
klund pushed a commit to branch support/1.12
in repository https://gitbox.apache.org/repos/asf/geode.git
commit 5a176a9df7ba8e169675e5a9ebb4527669d63589
Author: Kirk Lund
AuthorDate: Tue Sep 21 09:39:48 2021 -0700
GEODE-9486: Improve AnalyzeSerializables integration tests (#6878)
Allow geode modules to have an AnalyzeSerializables integration
test without implementing SanctionedSerializablesService.
Improve debugging information for AnalyzeSerializables integration
tests:
- Provide new failure message when no SanctionedSerializablesService
exists.
- Create ANALYZE_SERIALIZABLES.md to provide detailed instructions for
failures involving AnalyzeSerializables integration tests.
- Reference ANALYZE_SERIALIZABLES.md in failure messages.
Remove geode-serialization and geode-common jars from geode-pulse
.war file:
- Change getModuleClass() to return Optional.
- Delete PulseSanctionedSerializablesService and its resources.
- Change geode-pulse dependency on geode-serialization to be for
integrationTest only.
(cherry picked from commit 7bd1d73dcd02712a10e5c3f2ac5ac0522923bf9e)
(cherry picked from commit 14582d05ed0032f5315313e3d29c1773b7bf0f53)
(cherry picked from commit c685f06f64b8c7d9d160201db1b99597e53ce9d9)
---
...lyzeConnectorsSerializablesIntegrationTest.java | 6 +-
.../AnalyzeCoreSerializablesIntegrationTest.java | 6 +-
.../AnalyzeCQSerializablesIntegrationTest.java | 6 +-
.../AnalyzeDUnitSerializablesIntegrationTest.java | 4 +-
.../AnalyzeGfshSerializablesIntegrationTest.java | 6 +-
.../AnalyzeJUnitSerializablesIntegrationTest.java | 5 +-
.../AnalyzeDataSerializablesJUnitTestBase.java | 55 +-
.../AnalyzeSerializablesJUnitTestBase.java | 71 ++---
.../AnalyzeLuceneSerializablesIntegrationTest.java | 6 +-
...lyzeManagementSerializablesIntegrationTest.java | 6 +-
...lyzeMembershipSerializablesIntegrationTest.java | 5 +-
...alyzeMemcachedSerializablesIntegrationTest.java | 6 +-
geode-pulse/build.gradle | 2 +
.../AnalyzePulseSerializablesIntegrationTest.java | 7 +-
...ctionedSerializablesServiceIntegrationTest.java | 39 ---
.../PulseSanctionedSerializablesService.java | 27 -
...al.serialization.SanctionedSerializablesService | 15 ---
.../sanctioned-geode-pulse-serializables.txt | 0
.../AnalyzeRedisSerializablesIntegrationTest.java | 6 +-
geode-serialization/ANALYZE_SERIALIZABLES.md | 116 +
...eSerializationSerializablesIntegrationTest.java | 6 +-
.../AnalyzeWANSerializablesIntegrationTest.java| 6 +-
.../AnalyzeWebApiSerializablesIntegrationTest.java | 6 +-
.../sanctioned-geode-web-api-serializables.txt | 0
24 files changed, 254 insertions(+), 158 deletions(-)
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 b53ac7d..a5fa5a1 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
@@ -14,6 +14,8 @@
*/
package org.apache.geode.codeAnalysis;
+import java.util.Optional;
+
import org.junit.experimental.categories.Category;
import
org.apache.geode.connectors.jdbc.internal.ConnectorsSanctionedSerializablesService;
@@ -29,7 +31,7 @@ public class AnalyzeConnectorsSerializablesIntegrationTest
}
@Override
- protected Class getModuleClass() {
-return ConnectorsSanctionedSerializablesService.class;
+ protected Optional> getModuleClass() {
+return Optional.of(ConnectorsSanctionedSerializablesService.class);
}
}
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 54128fe..6832442 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
@@ -14,6 +14,8 @@
*/
package org.apache.geode.codeAnalysis;
+import java.util.Optional;
+
import org.junit.experimental.categories.Category;
import org.apache.geode.internal.CoreSanctionedSerializablesService;
@@ -28,7 +30,7 @@ public class AnalyzeCoreSerializablesIntegrationTest extends
AnalyzeSerializable
}
@Override
- protected Class getModuleClass() {
-return
[geode] branch support/1.12 updated (e4aee56 -> 5a176a9)
This is an automated email from the ASF dual-hosted git repository. klund pushed a change to branch support/1.12 in repository https://gitbox.apache.org/repos/asf/geode.git. from e4aee56 GEODE-9655: Bump shiro-core from 1.7.1 to 1.8.0 new 5c4b7e1 GEODE-9486: Fix validate-serializable-objects (#6823) new 5a176a9 GEODE-9486: Improve AnalyzeSerializables integration tests (#6878) The 2 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: .../geode/test/junit/rules/GeodeDevRestClient.java | 5 +- .../internal/web/RestRegionAPIIntegrationTest.java | 514 + ...yzeConnectorsSerializablesIntegrationTest.java} | 13 +- ...ctionedSerializablesServiceIntegrationTest.java | 24 +- .../ConnectorsSanctionedSerializablesService.java | 16 +- .../ConnectorsDistributedSystemService.java| 41 -- ...e.distributed.internal.DistributedSystemService | 16 - ...al.serialization.SanctionedSerializablesService | 15 + .../sanctioned-geode-connectors-serializables.txt | 0 ...> AnalyzeCoreSerializablesIntegrationTest.java} | 16 +- ...tionedSerializablesServiceIntegrationTest.java} | 23 +- .../internal/InternalDistributedSystem.java| 25 +- .../CoreSanctionedSerializablesService.java| 16 +- .../geode/internal/InternalDataSerializer.java | 103 +++-- .../internal/ObjectInputStreamFilterWrapper.java | 44 +- ...al.serialization.SanctionedSerializablesService | 15 + ...lDataSerializerSerializationAcceptlistTest.java | 148 +++--- ...ctionedSerializablesServiceIntegrationTest.java | 23 +- ... => AnalyzeCQSerializablesIntegrationTest.java} | 13 +- .../apache/geode/codeAnalysis/excludedClasses.txt | 2 - .../org/apache/geode/codeAnalysis/openBugs.txt | 21 - .../cq/internal/CQDistributedSystemService.java| 43 -- .../internal/CQSanctionedSerializablesService.java | 16 +- ...e.distributed.internal.DistributedSystemService | 16 - ...al.serialization.SanctionedSerializablesService | 15 + .../AnalyzeDUnitSerializablesIntegrationTest.java | 63 +++ ...ctionedSerializablesServiceIntegrationTest.java | 23 +- .../apache/geode/codeAnalysis/excludedClasses.txt | 4 + .../codeAnalysis/sanctionedDataSerializables.txt | 47 ++ .../cli/commands/QueryCommandDUnitTestBase.java| 395 +--- .../DUnitSanctionedSerializablesService.java | 16 +- .../junit/rules/LocatorLauncherStartupRule.java| 3 +- .../junit/rules/ServerLauncherStartupRule.java | 3 +- ...al.serialization.SanctionedSerializablesService | 15 + .../sanctioned-geode-dunit-serializables.txt | 182 .../cli/commands/QueryCommandDUnitTest.java| 23 + .../AnalyzeGfshSerializablesIntegrationTest.java | 16 +- ...ctionedSerializablesServiceIntegrationTest.java | 56 +++ .../apache/geode/codeAnalysis/excludedClasses.txt | 2 - .../org/apache/geode/codeAnalysis/openBugs.txt | 18 - .../internal/GfshDistributedSystemService.java | 43 -- .../GfshSanctionedSerializablesService.java| 16 +- ...e.distributed.internal.DistributedSystemService | 16 - ...al.serialization.SanctionedSerializablesService | 15 + geode-junit/build.gradle | 3 + .../AnalyzeJUnitSerializablesIntegrationTest.java | 28 +- ...ctionedSerializablesServiceIntegrationTest.java | 24 +- .../apache/geode/codeAnalysis/excludedClasses.txt | 3 + .../codeAnalysis/sanctionedDataSerializables.txt | 99 .../AnalyzeDataSerializablesJUnitTestBase.java | 387 .../AnalyzeSerializablesJUnitTestBase.java | 485 ++- .../geode/codeAnalysis/CompiledClassUtils.java | 2 + ...nedSerializablesServiceIntegrationTestBase.java | 89 .../categories/SanctionedSerializablesTest.java| 12 +- .../JUnitSanctionedSerializablesService.java | 16 +- .../geode/test/junit/rules/ClassAnalysisRule.java | 136 ++ .../geode/test/junit/rules/ConcurrencyRule.java| 4 +- .../apache/geode/test/junit/rules/GsRandom.java| 34 +- .../apache/geode/test/junit/rules/RandomRule.java | 311 + .../apache/geode/test/junit/rules/RuleList.java| 16 +- .../rules/serializable/SerializableTimeout.java| 38 +- .../org/apache/geode/test/util/ResourceUtils.java | 37 +- ...al.serialization.SanctionedSerializablesService | 15 + .../sanctioned-geode-junit-serializables.txt | 111 + .../geode/test/junit/rules/RandomRuleTest.java | 123 + .../serializable/SerializableTimeoutTest.java | 26 +- ...ctionedSerializablesServiceIntegrationTest.java | 23 +- ...AnalyzeLuceneSerializablesIntegrationTest.java} | 14 +- .../apache/geode/codeAnalysis/excludedClasses.txt | 2 - .../org/apache/geode/codeAnalysis/openBugs.txt |
[geode] branch develop updated (fc75182 -> 28321af)
This is an automated email from the ASF dual-hosted git repository.
heybales pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git.
from fc75182 GEODE-4181: Add JUnit 5 Support (#6911)
add 28321af GEODE-9516: add ZINTERSTORE command (#6861)
No new revisions were added by this update.
Summary of changes:
...a => ZInterStoreNativeRedisAcceptanceTest.java} | 2 +-
.../server/AbstractHitsMissesIntegrationTest.java | 6 +
.../AbstractZInterStoreIntegrationTest.java| 774 +
.../AbstractZUnionStoreIntegrationTest.java| 145 ++--
...onTest.java => ZInterStoreIntegrationTest.java} | 2 +-
.../geode/redis/internal/RedisCommandType.java | 2 +
.../geode/redis/internal/RedisConstants.java | 2 +
.../redis/internal/data/NullRedisSortedSet.java| 3 +-
.../geode/redis/internal/data/RedisSortedSet.java | 91 ++-
.../RedisSortedSetCommandsFunctionExecutor.java| 25 +-
.../executor/sortedset/RedisSortedSetCommands.java | 2 +
.../internal/executor/sortedset/ZAggregator.java | 10 +-
...ZRankExecutor.java => ZInterStoreExecutor.java} | 14 +-
.../internal/executor/sortedset/ZKeyWeight.java| 6 +-
...UnionStoreExecutor.java => ZStoreExecutor.java} | 110 ++-
.../executor/sortedset/ZUnionStoreExecutor.java| 105 +--
.../redis/internal/data/RedisSortedSetTest.java| 2 +-
17 files changed, 1040 insertions(+), 261 deletions(-)
copy
geode-apis-compatible-with-redis/src/acceptanceTest/java/org/apache/geode/redis/internal/executor/sortedset/{ZPopMaxNativeRedisAcceptanceTest.java
=> ZInterStoreNativeRedisAcceptanceTest.java} (92%)
create mode 100755
geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/sortedset/AbstractZInterStoreIntegrationTest.java
copy
geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/sortedset/{ZUnionStoreIntegrationTest.java
=> ZInterStoreIntegrationTest.java} (94%)
copy
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/executor/sortedset/{ZRankExecutor.java
=> ZInterStoreExecutor.java} (75%)
copy
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/executor/sortedset/{ZUnionStoreExecutor.java
=> ZStoreExecutor.java} (51%)
[geode] branch develop updated (468e0a3 -> fc75182)
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 468e0a3 GEODE-9581: Fix possible sending of duplicate events (#6896) add fc75182 GEODE-4181: Add JUnit 5 Support (#6911) No new revisions were added by this update. Summary of changes: .../src/test/resources/expected-pom.xml| 20 ++ .../gradle/plugins/DependencyConstraints.groovy| 11 + extensions/geode-modules/build.gradle | 1 + geode-apis-compatible-with-redis/build.gradle | 3 + geode-common/build.gradle | 2 + geode-concurrency-test/build.gradle| 1 + .../geode/test/dunit/internal/ProcessManager.java | 1 + geode-jmh/build.gradle | 1 + geode-junit/build.gradle | 5 + .../test/junit/runners/GeodeParamsRunner.java | 143 .../test/junit/rules/ConcurrencyRuleTest.java | 8 +- .../test/junit/runners/ComparableDescription.java | 89 + .../test/junit/runners/GeodeParamsRunnerTest.java | 359 + geode-junit/src/test/resources/expected-pom.xml| 20 ++ geode-rebalancer/build.gradle | 1 + geode-web-management/build.gradle | 4 + gradle/jmh.gradle | 4 + gradle/test.gradle | 23 +- static-analysis/pmd-rules/build.gradle | 1 + 19 files changed, 684 insertions(+), 13 deletions(-) create mode 100644 geode-junit/src/test/java/org/apache/geode/test/junit/runners/ComparableDescription.java create mode 100644 geode-junit/src/test/java/org/apache/geode/test/junit/runners/GeodeParamsRunnerTest.java
[geode] branch develop updated (2744ab8 -> 468e0a3)
This is an automated email from the ASF dual-hosted git repository. alberto pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git. from 2744ab8 GEODE-9495: limit thread sleep in pub sub native redis acceptance test (#6899) add 468e0a3 GEODE-9581: Fix possible sending of duplicate events (#6896) No new revisions were added by this update. Summary of changes: .../cache/wan/serial/SerialGatewaySenderQueue.java | 23 - .../serial/SerialGatewaySenderQueueJUnitTest.java | 101 ++--- 2 files changed, 87 insertions(+), 37 deletions(-)
