Register abs-expire module APIs#11025
Conversation
|
i guess it also need to backport to 6.2 |
|
seen this kind of error a few times before.. |
yes, i think we can do it, like assert(REDISMODULE_GET_API(SetAbsExpire) == REDISMODULE_OK); i guess we need it, i did it and i found that we have an extra Fortunately nothing else was missed, but doing this require a lot of changes... |
|
the assertion can be part of the macro. |
Prevent forgetting to register API, adding assertions, see redis#11025 Also remove `ReplySetPushLength`, it is a dead code.
|
ok, i am doing it, in #11033 |
RM_SetAbsExpire and RM_GetAbsExpire were not actually operational since they were introduced, due to omission in API registration. (cherry picked from commit 39d216a)
RM_SetAbsExpire and RM_GetAbsExpire were not actually operational since they were introduced, due to omission in API registration. (cherry picked from commit 39d216a)
RM_SetAbsExpire and RM_GetAbsExpire were not actually operational since they were introduced, due to omission in API registration.
RM_SetAbsExpire and RM_GetAbsExpire were not actually operational since they were introduced, due to omission in API registration.
Sorry this was an omission in the previous PR #8564