Darrel Schneider created GEODE-9904:
---------------------------------------

             Summary: RedisSet spop and srandmember should not copy entire 
MemberSet
                 Key: GEODE-9904
                 URL: https://issues.apache.org/jira/browse/GEODE-9904
             Project: Geode
          Issue Type: Improvement
          Components: redis
            Reporter: Darrel Schneider


The current implementation of RedisSet spop and srandmember always copy the 
entire MemberSet. So if you have a redis set with a million items, and than ask 
for one random one, instead of it reading just one of the items out of the set 
it first copies the entire set.

The call that makes the copy is "members.toArray". It should be pretty easy to 
add spop and srandmember support to the MemberSet super class. Since MemberSet 
uses a flat array to store its items it should work well with supporting 
indexing of its array to get a random element. 



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to