[geode] branch develop updated (c246736345 -> de91e33052)
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 c246736345 GEODE-10053: Use stored DistributionConfig to create SSL configurations (#7366) add de91e33052 GEODE-10216: Add test for existence of VM stats (#7551) No new revisions were added by this update. Summary of changes: .../statistics/StatisticsDistributedTest.java | 188 + 1 file changed, 113 insertions(+), 75 deletions(-)
[geode-site] branch asf-site updated: GEODE-10205: Correct Java API method from getDiskStore to findDiskStore (#16)
This is an automated email from the ASF dual-hosted git repository.
dbarnes pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/geode-site.git
The following commit(s) were added to refs/heads/asf-site by this push:
new 924d3555 GEODE-10205: Correct Java API method from getDiskStore to
findDiskStore (#16)
924d3555 is described below
commit 924d3555c8265b8c5e2a7ad93547ba6f12f73109
Author: Max Hufnagel
AuthorDate: Tue Apr 5 16:34:28 2022 -0700
GEODE-10205: Correct Java API method from getDiskStore to findDiskStore
(#16)
---
docs/guide/114/managing/disk_storage/compacting_disk_stores.html | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/guide/114/managing/disk_storage/compacting_disk_stores.html
b/docs/guide/114/managing/disk_storage/compacting_disk_stores.html
index 0c9c3a59..0a9f63ae 100644
--- a/docs/guide/114/managing/disk_storage/compacting_disk_stores.html
+++ b/docs/guide/114/managing/disk_storage/compacting_disk_stores.html
@@ -2708,7 +2708,7 @@ See Disk Sto
Compact the logs for a single online disk store through the API, with
the forceCompaction method. This method first rolls the oplogs and
then compacts them. Example:
-myCache.getDiskStore("myDiskStore").forceCompaction();
+myCache.findDiskStore("myDiskStore").forceCompaction();
Using gfsh, compact a disk store with the compact
disk-store command. Examples:
gfshcompact disk-store --name=Disk1
[geode] branch support/1.12 updated: GEODE-9142: increase heavy-lifter disk to 200GB to match 1.13, 1.14 and develop
This is an automated email from the ASF dual-hosted git repository.
onichols pushed a commit to branch support/1.12
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/support/1.12 by this push:
new 73f1fb935a GEODE-9142: increase heavy-lifter disk to 200GB to match
1.13, 1.14 and develop
73f1fb935a is described below
commit 73f1fb935ad929a9cf7e46f295569f9c5f336153
Author: Owen Nichols
AuthorDate: Tue Apr 5 14:28:35 2022 -0700
GEODE-9142: increase heavy-lifter disk to 200GB to match 1.13, 1.14 and
develop
(cherry picked from commit a84c26c7610a56b6e22039faf6a8858393e4124a)
---
ci/scripts/create_instance.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ci/scripts/create_instance.sh b/ci/scripts/create_instance.sh
index d6df95ce21..e26671e6c9 100755
--- a/ci/scripts/create_instance.sh
+++ b/ci/scripts/create_instance.sh
@@ -141,7 +141,7 @@ INSTANCE_INFORMATION=$(gcloud compute
--project=${GCP_PROJECT} instances create
--network="${GCP_NETWORK}" \
--subnet="${GCP_SUBNETWORK}" \
--image="${IMAGE_NAME}" \
- --boot-disk-size=100GB \
+ --boot-disk-size=200GB \
--boot-disk-type=pd-ssd \
--labels="${LABELS}" \
--tags="heavy-lifter" \
[geode] branch develop updated (f1d5a25877 -> c246736345)
This is an automated email from the ASF dual-hosted git repository. nnag pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/geode.git from f1d5a25877 GEODE-10198: Fix LINSERT's case-sensitivity for arguments (#7552) add c246736345 GEODE-10053: Use stored DistributionConfig to create SSL configurations (#7366) No new revisions were added by this update. Summary of changes: .../LocatorLauncherLocalIntegrationTest.java | 22 .../apache/geode/distributed/LocatorLauncher.java | 58 +- 2 files changed, 67 insertions(+), 13 deletions(-)
[geode-native] branch develop updated: Fix Ubuntu build break due to cmake upgrade (#952)
This is an automated email from the ASF dual-hosted git repository.
bbender 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 107d317ac Fix Ubuntu build break due to cmake upgrade (#952)
107d317ac is described below
commit 107d317acde1bd4fadf5ea9eea31d5a4a0a31f45
Author: Blake Bender
AuthorDate: Tue Apr 5 14:37:34 2022 -0700
Fix Ubuntu build break due to cmake upgrade (#952)
- New cmake version changes treatment of CMAKE_INSTALL_LIBDIR variable,
which now evaluates to different things for xerces build and main
client build. As a result, we build Xerces, then go looking to link
to it in the wrong place. It appears cmake has special code on Ubuntu
to use the wrong lib dir
iff CMAKE_INSTALL_PREFIX is set to /usr or /usr/local, so setting it to
something else seems the safest approach.
---
ci/base/base.yml | 6 ++
1 file changed, 6 insertions(+)
diff --git a/ci/base/base.yml b/ci/base/base.yml
index 363350675..34386ea2a 100644
--- a/ci/base/base.yml
+++ b/ci/base/base.yml
@@ -96,9 +96,15 @@ builds:
- _: #@ template.replace(new_build("ubuntu-18.04"))
image_family: build-ubuntu-18-04
source_image_family: ubuntu-1804-lts
+#@yaml/map-key-override
+params:
+ CMAKE_CONFIGURE_FLAGS: "-DCMAKE_INSTALL_PREFIX=/home/build"
- _: #@ template.replace(new_build("ubuntu-20.04"))
image_family: build-ubuntu-20-04
source_image_family: ubuntu-2004-lts
+#@yaml/map-key-override
+params:
+ CMAKE_CONFIGURE_FLAGS: "-DCMAKE_INSTALL_PREFIX=/home/build"
configs: [ ]
[geode] branch support/1.13 updated: GEODE-9142: increase heavy-lifter disk to 200GB to match 1.14 and develop
This is an automated email from the ASF dual-hosted git repository.
onichols pushed a commit to branch support/1.13
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/support/1.13 by this push:
new a84c26c761 GEODE-9142: increase heavy-lifter disk to 200GB to match
1.14 and develop
a84c26c761 is described below
commit a84c26c7610a56b6e22039faf6a8858393e4124a
Author: Owen Nichols
AuthorDate: Tue Apr 5 14:28:35 2022 -0700
GEODE-9142: increase heavy-lifter disk to 200GB to match 1.14 and develop
---
ci/scripts/create_instance.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ci/scripts/create_instance.sh b/ci/scripts/create_instance.sh
index ed72a6a9a6..7ddbb31a39 100755
--- a/ci/scripts/create_instance.sh
+++ b/ci/scripts/create_instance.sh
@@ -141,7 +141,7 @@ INSTANCE_INFORMATION=$(gcloud compute
--project=${GCP_PROJECT} instances create
--network="${GCP_NETWORK}" \
--subnet="${GCP_SUBNETWORK}" \
--image="${IMAGE_NAME}" \
- --boot-disk-size=100GB \
+ --boot-disk-size=200GB \
--boot-disk-type=pd-ssd \
--labels="${LABELS}" \
--tags="heavy-lifter" \
[geode] branch develop updated: GEODE-10198: Fix LINSERT's case-sensitivity for arguments (#7552)
This is an automated email from the ASF dual-hosted git repository.
jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new f1d5a25877 GEODE-10198: Fix LINSERT's case-sensitivity for arguments
(#7552)
f1d5a25877 is described below
commit f1d5a2587754e49f2c7e6f0e1fc39b0221fb8ec3
Author: Bala Kaza Venkata <43208983+balak...@users.noreply.github.com>
AuthorDate: Tue Apr 5 14:16:32 2022 -0400
GEODE-10198: Fix LINSERT's case-sensitivity for arguments (#7552)
LINSERT was case-sensitive with its arguments BEFORE/AFTER.Redis
implementation of LINSERT is not case-sensitive.
---
.../resources/0001-configure-redis-tests.patch | 70 ++
.../commands/executor/list/LInsertExecutor.java| 4 +-
2 files changed, 19 insertions(+), 55 deletions(-)
diff --git
a/geode-for-redis/src/acceptanceTest/resources/0001-configure-redis-tests.patch
b/geode-for-redis/src/acceptanceTest/resources/0001-configure-redis-tests.patch
index 8ea96fe42f..6bf8dbd05f 100644
---
a/geode-for-redis/src/acceptanceTest/resources/0001-configure-redis-tests.patch
+++
b/geode-for-redis/src/acceptanceTest/resources/0001-configure-redis-tests.patch
@@ -1362,13 +1362,13 @@ index b5bd48cb0..5b9a529c7 100644
r linsert key $where $otherele $ele
}
diff --git a/tests/unit/type/list.tcl b/tests/unit/type/list.tcl
-index 1557082a2..5395920c7 100644
+index 1557082a2..3d6a3ace4 100644
--- a/tests/unit/type/list.tcl
+++ b/tests/unit/type/list.tcl
@@ -85,130 +85,130 @@ start_server {
assert_encoding quicklist $key
}
-
+
-foreach {type large} [array get largevalue] {
-test "BLPOP, BRPOP: single existing list - $type" {
-set rd [redis_deferring_client]
@@ -1551,12 +1551,12 @@ index 1557082a2..5395920c7 100644
+ # r lpush list b
+ # $rd read
+ # } {list b}
-
+
test "BLPOP with same key multiple times should work (issue #801)" {
set rd [redis_deferring_client]
-r del list1 list2
+r del "{tag}list1" "{tag}list2"
-
+
# Data arriving after the BLPOP.
-$rd blpop list1 list2 list2 list1 0
-r lpush list1 a
@@ -1570,7 +1570,7 @@ index 1557082a2..5395920c7 100644
+$rd blpop "{tag}list1" "{tag}list2" "{tag}list2" "{tag}list1" 0
+r lpush "{tag}list2" b
+assert_equal [$rd read] {{{tag}list2} b}
-
+
# Data already there.
-r lpush list1 a
-r lpush list2 b
@@ -1610,13 +1610,13 @@ index 1557082a2..5395920c7 100644
+ # r exec
+ # $rd read
+ # } {list c}
-
+
test "BLPOP with variadic LPUSH" {
set rd [redis_deferring_client]
@@ -222,203 +222,204 @@ start_server {
assert_equal foo [lindex [r lrange blist 0 -1] 0]
}
-
+
-test "BRPOPLPUSH with zero timeout should block indefinitely" {
-set rd [redis_deferring_client]
-r del blist target
@@ -2018,7 +2018,7 @@ index 1557082a2..5395920c7 100644
@@ -434,11 +435,11 @@ start_server {
assert_error "ERR*is negative*" {$rd read}
}
-
+
-test "$pop: with non-integer timeout" {
-set rd [redis_deferring_client]
-$rd $pop blist1 1.1
@@ -2029,11 +2029,11 @@ index 1557082a2..5395920c7 100644
+#$rd $pop blist1 1.1
+#assert_error "ERR*not an integer*" {$rd read}
+#}
-
+
test "$pop: with zero timeout should block indefinitely" {
# To test this, use a timeout of 0 and wait a second.
-@@ -452,83 +453,84 @@ start_server {
-
+@@ -452,65 +453,66 @@ start_server {
+
test "$pop: second argument is not a list" {
set rd [redis_deferring_client]
-r del blist1 blist2
@@ -2044,7 +2044,7 @@ index 1557082a2..5395920c7 100644
+$rd $pop "{tag}blist1" "{tag}blist2" 1
assert_error "WRONGTYPE*" {$rd read}
}
-
+
test "$pop: timeout" {
set rd [redis_deferring_client]
-r del blist1 blist2
@@ -2053,7 +2053,7 @@ index 1557082a2..5395920c7 100644
+$rd $pop "{tag}blist1" "{tag}blist2" 1
assert_equal {} [$rd read]
}
-
+
test "$pop: arguments are empty" {
set rd [redis_deferring_client]
-r del blist1 blist2
@@ -2124,7 +2124,7 @@ index 1557082a2..5395920c7 100644
+ # assert_equal 0 [r rpushx xlist a]
+ # assert_equal 0 [r llen xlist]
+ # }
-
+
foreach {type large} [array get largevalue] {
-test "LPUSHX, RPUSHX - $type" {
-create_list xlist "$large c"
@@ -2134,24 +2134,6 @@ index 1557082a2..5395920c7 100644
-assert_equal 9 [r lpushx xlist y3 y2 y1]
-assert_equal "y1 y2 y3 a $large c d 42 x" [r lrange xlist 0 -1]
-}
--
--
[geode] branch develop updated: GEODE-10126: Replace Geode for Redis properties with system properties (#7478)
This is an automated email from the ASF dual-hosted git repository.
jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new ee1c412d8a GEODE-10126: Replace Geode for Redis properties with system
properties (#7478)
ee1c412d8a is described below
commit ee1c412d8af3e6311f569948c3248eddc1fe8009
Author: Jens Deppe
AuthorDate: Tue Apr 5 09:53:00 2022 -0700
GEODE-10126: Replace Geode for Redis properties with system properties
(#7478)
* GEODE-10126: Replace Geode for Redis properties with system properties
- All previous Geode for Redis properties that were defined in
geode-core are replaced by java system properties.
- gemfire.geode-for-redis-port
- gemfire.geode-for-redis-bind-address
- gemfire.geode-for-redis-bind-username
- gemfire.geode-for-redis-redundant-copies
- gemfire.geode-for-redis-enabled
- Redis is enabled (using defaults) if the `enabled` property is set to
true OR if any of the other properties are explicitly set as java
system properties.
- Introduce `RedisConfiguration` interface to access properties.
- Concrete implementation of `SystemPropertyBasedRedisConfiguration`
that uses system properties to define Radish properties.
- Integration Tests use `TestRedisConfiguration` to avoid having to set
system properties.
- Services started by Geode (`CacheService`s) that throw exceptions
during initialization will cause the Cache to fail to startup.
Previously any such errors were only logged.
- Default bind address is explicitly 0.0.0.0 which is more expressive than
just an empty string
---
.../geode/distributed/ConfigurationProperties.java | 77 --
.../internal/AbstractDistributionConfig.java | 26
.../distributed/internal/DistributionConfig.java | 94
.../internal/DistributionConfigImpl.java | 80 +-
.../geode/internal/cache/GemFireCacheImpl.java | 1 +
.../internal/DistributionConfigImplTest.java | 8 +-
.../GeodeRedisServerStartupAcceptanceTest.java | 139 ++---
...eRedisServerStartupUsingGfshAcceptanceTest.java | 58 ---
.../apache/geode/redis/GeodeRedisServerRule.java | 14 +-
.../apache/geode/redis/TestRedisConfiguration.java | 97
.../test/dunit/rules/RedisClusterStartupRule.java | 18 +--
.../executor/hash/HashesAndCrashesDUnitTest.java | 6 +-
.../commands/executor/key/RenameDUnitTest.java | 6 +-
.../commands/executor/list/RPushDUnitTest.java | 4 +-
.../commands/executor/sortedset/ZRemDUnitTest.java | 4 +-
.../sortedset/ZRemRangeByRankDUnitTest.java| 8 +-
.../commands/executor/string/MSetDUnitTest.java| 6 +-
.../commands/executor/string/MSetNXDUnitTest.java | 6 +-
.../auth/AuthWhileServersRestartDUnitTest.java | 4 +-
.../redis/session/SessionsAndCrashesDUnitTest.java | 4 +-
.../executor/connection/AuthIntegrationTest.java | 26 ++--
.../geode/redis/internal/GeodeRedisServer.java | 23 +--
.../geode/redis/internal/GeodeRedisService.java| 25 ++-
.../geode/redis/internal/RedisConfiguration.java | 59 +++
.../SystemPropertyBasedRedisConfiguration.java | 170 +
.../redis/internal/netty/NettyRedisServer.java | 15 +-
.../redis/internal/services/RegionProvider.java| 10 +-
27 files changed, 540 insertions(+), 448 deletions(-)
diff --git
a/geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
b/geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
index d8d32565a6..1a4e205e2e 100644
---
a/geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
+++
b/geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
@@ -1920,83 +1920,6 @@ public interface ConfigurationProperties {
* Since: Geode 1.0
*/
String OFF_HEAP_MEMORY_SIZE = "off-heap-memory-size";
- /**
- * The static String definition of the "geode-for-redis-bind-address"
property
- *
- * Description: Specifies the address on which Geode for Redis is
- * listening. If set to the empty string or this property is not specified,
the server listens on
- * all local addresses.
- *
- * Default: ""
- *
- * Since: Geode 1.15
- */
- String GEODE_FOR_REDIS_BIND_ADDRESS = "geode-for-redis-bind-address";
- /**
- * The static String definition of the "geode-for-redis-enabled"
property
- *
- * Description: When the default value of false, Geode for Redis is
- * not available.
- * Set to true to enable Geode for Redis.
- *
- * Default: false
- * When the default value of false, Geode for Redis is not available.
- * Set to true to enable Geode for Redis.
- *
- * Since: Geode 1.15
- */
- String GEODE_FOR_REDIS_ENABLED
