upthewaterspout commented on a change in pull request #6461:
URL: https://github.com/apache/geode/pull/6461#discussion_r631951647
##########
File path:
geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/hash/AbstractHScanIntegrationTest.java
##########
@@ -453,7 +453,7 @@ public void should_notErrorGivenNonzeroCursorOnFirstCall() {
ScanResult<Map.Entry<String, String>> result = jedis.hscan("colors", "5");
assertThat(new HashSet<>(result.getResult()))
- .containsExactlyInAnyOrderElementsOf(data.entrySet());
+ .isSubsetOf(data.entrySet());
Review comment:
The behavior of hscan when you start with a non-zero cursor is
undefined, except that it should not return an error. So we can only reasonably
expect that we might see some but not necessarily all of the elements of the
hash.
--
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:
[email protected]
