GitHub user Aetherance added a comment to the discussion: Proposal: Keyspace 
Notifications for Kvrocks

> Because Kvrocks uses a different expiration mechanism from Redis, I don’t 
> think `expired` events can be implemented correctly on our side. Therefore, I 
> don’t think we need to support this event.

I agree with this. `expired` should stay out of scope for this proposal. 
Kvrocks handles expiration differently from Redis, so it’s hard to provide 
correct Redis-compatible `expired` events, especially on replicas. I’ll keep 
this MVP focused on `set` and `del` only as requested in issue #2915. Whether 
to support other commands/events can be discussed separately later.

GitHub link: 
https://github.com/apache/kvrocks/discussions/3533#discussioncomment-17435848

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to