Skip to content

add NOTIFY_MODULE flag#6963

Closed
ashtul wants to merge 1 commit intoredis:unstablefrom
ashtul:add-NOTIFY_MODULE-flag
Closed

add NOTIFY_MODULE flag#6963
ashtul wants to merge 1 commit intoredis:unstablefrom
ashtul:add-NOTIFY_MODULE-flag

Conversation

@ashtul
Copy link
Contributor

@ashtul ashtul commented Mar 8, 2020

Add a generic flag to be used by modules.

@antirez
Copy link
Contributor

antirez commented Mar 13, 2020

Please could you elaborate? I don't see any change in the APIs but just the additional flag.

Copy link

@ProBeta1 ProBeta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes in API are required .

@ashtul
Copy link
Contributor Author

ashtul commented Mar 18, 2020

This is meant for use by Redis module developers.
Currently, the REDISMODULE_NOTIFY_GENERIC flag is the one that makes the most sense but it will raise many additional events.

@yossigo yossigo mentioned this pull request Dec 8, 2020
52 tasks
@ashtul ashtul closed this Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants