[ https://issues.apache.org/jira/browse/GEODE-8864?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17273069#comment-17273069 ]
ASF GitHub Bot commented on GEODE-8864: --------------------------------------- jhutchison commented on a change in pull request #5954: URL: https://github.com/apache/geode/pull/5954#discussion_r565530655 ########## File path: geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHScanIntegrationTest.java ########## @@ -336,48 +418,31 @@ public void givenMultipleCountsAndMatches_returnsEntriesMatchingLastMatchParamet } @Test - public void givenNegativeCursor_returnsEntriesUsingAbsoluteValueOfCursor() { + public void should_notReturnValue_givenValueWasRemovedBeforeHSCANISCalled() + throws ExecutionException, InterruptedException { + Map<String, String> entryMap = new HashMap<>(); entryMap.put("1", "yellow"); - entryMap.put("2", "green"); - entryMap.put("3", "orange"); + entryMap.put("12", "green"); + entryMap.put("3", "grey"); jedis.hmset("colors", entryMap); + Future deleteFuture = doAsyncHdel("colors", "3"); + deleteFuture.get(); Review comment: Yeah, I see your point. Let me look at what I thought I was doing here... ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > finish implementation of Redis HScan Command > -------------------------------------------- > > Key: GEODE-8864 > URL: https://issues.apache.org/jira/browse/GEODE-8864 > Project: Geode > Issue Type: New Feature > Components: redis > Reporter: John Hutchison > Priority: Major > Labels: pull-request-available > -- This message was sent by Atlassian Jira (v8.3.4#803005)