[GitHub] [kvrocks] zncleon commented on pull request #1768: Add the support of the BF.INSERT command

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

zncleon commented on PR #1768:
URL: https://github.com/apache/kvrocks/pull/1768#issuecomment-1725175080

   The reasons Why I change the msg are:
   1. If I direct parse (e.g. capacity) in `BF.Reserve` and use parser's `take` and `GET_OR_RET` in `BF.INSERT`, the error msg would be differentiated.
   2. Now the parse error msg is the same as the RedisBloom.


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