[GitHub] [incubator-kvrocks] jihuayu commented on issue #1460: Add the support of the ZRANGESTORE command

["jihuayu (via GitHub)" <gi...@apache.org>]

jihuayu commented on issue #1460:
URL: https://github.com/apache/incubator-kvrocks/issues/1460#issuecomment-1573826940

   In this command. I used `ZSet#RangeByLex` get members and use `ZSet#Add` add them to new zset.
   But `ZSet#Add` need MemberScore(member+score), `ZSet#RangeByLex` only return member.
   
   So I have 3 way to solve it.
   1. After I use `ZSet#RangeByLex` get members, use `ZSet#Score` to get it score. It may cost additional performance overhead.
   2. Add a new `ZSet#RangeByLex` function that return `MemberScore`. 
   3. Change the `ZSet#RangeByLex` function make it return MemberScore. I found we have already get the socre in `ZSet#RangeByLex` function , so return `MemberScore` will not incur additional performance overhead.
   
   I think 3rd is best. Is it OK?
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org