kirito632 commented on PR #3504: URL: https://github.com/apache/kvrocks/pull/3504#issuecomment-4756036317
I noticed the same UINT64_MAX issue also exists in some shared stream ID parsers (ParseStreamEntryID(), ParseRangeEnd(), ParseNextStreamEntryIDStrategy()). I intentionally limited this fix to the new XACKDEL parser because those shared helpers are used by several existing stream commands and changing them would significantly broaden the scope of this PR. I'm happy to address the shared parsers in a separate follow-up PR if needed. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
