You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by on...@apache.org on 2021/03/04 02:43:50 UTC

[geode] 04/04: GEODE-8624: Support Redis HINCRBYFLOAT command (#5986)

This is an automated email from the ASF dual-hosted git repository.

onichols pushed a commit to branch support/1.14
in repository https://gitbox.apache.org/repos/asf/geode.git

commit d201090a6ae6ef4d591e936ffce776b3f8919e89
Author: Jens Deppe <jd...@pivotal.io>
AuthorDate: Thu Feb 4 05:50:04 2021 -0800

    GEODE-8624: Support Redis HINCRBYFLOAT command (#5986)
    
    - Switch to using BigDecimal instead of double for increment value.
    - Improve HINCRBYFLOAT output accuracy for very large values.
    - Add concurrency tests.
    - Mark HINCRBYFLOAT as supported.
    
    (cherry-picked from 8a0dc1aae8636a44c4fa4d5d30dcbc0efafb4430)
---
 .../redis/internal/executor/hash/AbstractHashesIntegrationTest.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHashesIntegrationTest.java b/geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHashesIntegrationTest.java
index 96493a4..d7b606f 100755
--- a/geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHashesIntegrationTest.java
+++ b/geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHashesIntegrationTest.java
@@ -202,7 +202,7 @@ public abstract class AbstractHashesIntegrationTest implements RedisPortSupplier
   }
 
   @Test
-  public void testHMGet_givenWrongNumberOfArguments() {
+  public void testHMGet_givenTooFewArguments() {
     assertThatThrownBy(() -> jedis.sendCommand(Protocol.Command.HMGET))
         .hasMessage("ERR wrong number of arguments for 'hmget' command");
     assertThatThrownBy(() -> jedis.sendCommand(Protocol.Command.HMGET, "1"))
@@ -819,7 +819,7 @@ public abstract class AbstractHashesIntegrationTest implements RedisPortSupplier
   }
 
   @Test
-  public void testConcurrentHIncrByFloat_sameKeyPerClient() throws InterruptedException {
+  public void testConcurrentHIncrByFloat_sameKeyPerClient() {
     String key = "HSET_KEY";
     String field = "HSET_FIELD";