Skip to content

Add INCREX keyspace notification event#3303

Closed
sundb wants to merge 1 commit into
redis:mainfrom
sundb:increx-notification
Closed

Add INCREX keyspace notification event#3303
sundb wants to merge 1 commit into
redis:mainfrom
sundb:increx-notification

Conversation

@sundb
Copy link
Copy Markdown

@sundb sundb commented May 12, 2026

Note

Low Risk
Low risk: documentation-only change that adds INCREX to the keyspace notification event list with no runtime or behavioral impact.

Overview
Documents keyspace notification behavior for the INCREX command, including which incrby/incrbyfloat event is emitted and how expiration-related options (EX/PX/EXAT/PXAT/PERSIST) map to additional expire/persist/del events.

Reviewed by Cursor Bugbot for commit 6d9d663. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 12, 2026

CLA assistant check
All committers have signed the CLA.

@LiorKogan LiorKogan requested a review from dwdougherty May 12, 2026 06:57
@dwdougherty
Copy link
Copy Markdown
Collaborator

Hi @sundb. Please change the target of the PR to DOC-6334 (my 8.8 feature branch). It can't go to main.

Apart from that, LGTM and I'll approve and merge when the target is changed.

@dwdougherty dwdougherty self-assigned this May 12, 2026
@dwdougherty dwdougherty added do not merge yet oss Redis Open Source labels May 12, 2026
@sundb sundb closed this May 12, 2026
@dwdougherty
Copy link
Copy Markdown
Collaborator

If it's easier for you, I can cherry-pick your change and add it to my other INCREX ticket. I'll leave it up to you.

@sundb
Copy link
Copy Markdown
Author

sundb commented May 12, 2026

If it's easier for you, I can cherry-pick your change and add it to my other INCREX ticket. I'll leave it up to you.

sure. thx for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet oss Redis Open Source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants