Behaviour of Replication::aggregate
Author: szepeviktorCreated Jan 21, 2023Updated Feb 20, 2025
Labelsbug
Describe the bug Was a TODO in source code.
At least for now we will replicate the previous behaviour of skipping automatic aggregation when using the redis-sentinel backend because $nodes contains an array of sentinel servers instead of Redis servers and SentinelReplication already gets the list of sentinels in the first argument of its constructor. SentinelReplication::add() actually knows how to handle connections marked with role=sentinel in their parameters but relying on it would require an explicit role to be set by the user and I would like to avoid enforcing that for now.
Source: predis/predis