This repository was archived by the owner on Apr 26, 2024. It is now read-only.
Repository navigation
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
Enable Synapse to connect to redis logical DB #15009
Description
Currently for Redis Database for distributed deployment (with workers). The available config option are redis host, port, there is no option in configuration to specify logical redis DB. The dependency synapse is using "txredisapi" does support the connection to a logical redis DB. Is it possible to enable the same in Synapse configuration?
Activity
clokep commentedon Feb 7, 2023
That sounds reasonable -- we would certainly accept a patch adding support for this. It should just be threading through the new configuration with a default to whatever happens if you don't provide it.
waclaw66 commentedon Feb 11, 2023
@999lakhisidhu: https://redis.io/docs/manual/pubsub/
Pub/Sub has no relation to the key space. It was made to not interfere with it on any level, including database numbers.
999lakhisidhu commentedon Feb 11, 2023
@waclaw66 I agree for pub/sub it may not matter, if you are connecting with a logical keyspace or normal connection but here we are storing some key,value pairs as well(i have observed it in redis db connected to synapse)… so these key, values can be segregated as per logical db..
waclaw66 commentedon Feb 11, 2023
I haven't seen any key-value in redis. What kind of data are stored there?
999lakhisidhu commentedon Feb 11, 2023
`[test@cloud]$ redis-cli -h localhost -p 6379 -n 3
localhost:6379[3]> keys *
localhost:6379[3]> GET "cache_v1:event_to_prev_state_group:$Cr35pecVCiC43-IjNqFtWIFUblXU0-cTAwnOUAc6JD4"
"23"
localhost:6379[3]>
`
@waclaw66 this is the snapshot for key,value pairs stored in redis DB