[ https://issues.apache.org/jira/browse/GEODE-8864?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17293010#comment-17293010 ]
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_r584867625 ########## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/data/RedisHash.java ########## @@ -203,49 +279,122 @@ public int hstrlen(ByteArrayWrapper field) { return new ArrayList<>(hash.keySet()); } - public Pair<BigInteger, List<Object>> hscan(Pattern matchPattern, int count, BigInteger cursor) { - List<Object> returnList = new ArrayList<Object>(); - int size = hash.size(); - BigInteger beforeCursor = new BigInteger("0"); - int numElements = 0; - int i = -1; - for (Map.Entry<ByteArrayWrapper, ByteArrayWrapper> entry : hash.entrySet()) { - ByteArrayWrapper key = entry.getKey(); - ByteArrayWrapper value = entry.getValue(); - i++; - if (beforeCursor.compareTo(cursor) < 0) { - beforeCursor = beforeCursor.add(new BigInteger("1")); + public ImmutablePair<BigInteger, List<Object>> hscan(UUID clientID, Pattern matchPattern, + int count, + BigInteger cursorParameter) { + + int startCursor = cursorParameter.intValue(); Review comment: So @jdeppe-pivotal brought this up in standup the other day. The decision the team landed on was to make these choices about data types based on our implementation (e.g. what makes sense with java data types in geode) and to guard against data types that we don't support at the command level (send an error back to any client trying to pass a datatype that we don't support internally). I'll make sure that Hscan follows this convention. I'll also put in a story to make sure that we're documenting differences between what data types we accept and what native redis accepts. ---------------------------------------------------------------- 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)