[GitHub] [kvrocks] jihuayu commented on a diff in pull request #1489: Add Redis-compatible cursors for `SCAN` commands

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

jihuayu commented on code in PR #1489:
URL: https://github.com/apache/kvrocks/pull/1489#discussion_r1238253115


##########
src/server/server.h:
##########
@@ -84,6 +88,39 @@ struct ChannelSubscribeNum {
   size_t subscribe_num;
 };
 
+// CURSOR_DICT_SIZE must be 2^n where n < 16

Review Comment:
   Sorry, I've confused the maximum values of size, index, and uint_16 again.
   Thank you for your patience.



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