Hello, We added a new parameter to ndb_redis module. If set to 1 a "FLUSHALL" command is issued to the REDIS server when reconnecting to a server after a failed command. This way for each failed command the database is flushed.
This is useful when a connection to the REDIS server may become unstable and some commands are issued but the replies are not received (it may happen that the command has reached the REDIS server but the reply was lost). If for example we increment and decrement counters this may leave the db in an invalid state.
This can especially happen if the allowed_timeouts/disable_time parameters are configured, since the first commands may be issued and the next ones are never tried. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1125
-- Commit Summary --
* ndb_redis: add flush_db_on_reconnect parameter
-- File Changes --
M src/modules/ndb_redis/doc/ndb_redis_admin.xml (32) M src/modules/ndb_redis/ndb_redis_mod.c (2) M src/modules/ndb_redis/redis_client.c (5)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1125.patch https://github.com/kamailio/kamailio/pull/1125.diff
Thanks, I am going to merge it. But I will remove the db_ from the name of its new parameter be cause it is a bit long and the operations are expected to be specific to redis interaction, no need to make it very explicit.
Merged #1125.