Aetherance opened a new pull request, #3541:
URL: https://github.com/apache/kvrocks/pull/3541
Implement keyspace notifications for `SET` and `DEL`, compatible with Redis
`notify-keyspace-events`.
This PR adds the initial notification emitters and configuration support:
- Support `K`, `E`, `g`, `$`, and `A` flags, where `A` currently expands
to the implemented event classes `g$`.
- Emit `set` notifications only when `SET` actually applies a write,
including conditional `SET` variants.
- Emit `del` notifications only for keys that are actually deleted.
- Deduplicate repeated keys in a single `DEL`, so `DEL a a` deletes and
notifies once.
- Queue notifications inside `MULTI/EXEC` and publish them after a
successful commit.
- Map notification DB names correctly:
- default namespace uses DB `0`
- `redis-databases` namespaces map back to Redis DB indexes
Unsupported notification classes are rejected for now until their emitters
are implemented.
Ref Proposal: #3533
Tracking Issue: #2915
Assisted by Codex/GPT-5.5.
--
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]
