theweakgod commented on PR #10866: URL: https://github.com/apache/apisix/pull/10866#issuecomment-1919144643
> > Are you suggesting changing it to incrby or zincrby? However, changing to these also does not address the issue of needing to establish a connection for both rediscluster and redis. > > Okay I understand it now. > > > Are you suggesting changing it to incrby or zincrby > > Yes I am inclined towards using `incrby`. I don't think we need hashes here. Just prefix the key with plugin name to make it unique. In fact, I think redis key management is more difficult than hash key management. If a key needs to be created after a while, and the value is just an integer, then I think it will be a disaster for this db, because deleting these keys Sometimes, users need to unlink a lot of keys, and to delete the hash, they only need to unlink this key. This is my idea -- 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]
