[Tarantool-patches] [PATCH 1/7] replication: always send raft state to subscribers

[Cyrill Gorcunov]

On Thu, Jun 10, 2021 at 04:32:51PM +0300, Serge Petrenko wrote:
> -	if (raft_is_enabled(box_raft())) {
> +	if (replica_version_id >= version_id(2, 6, 0) && !anon) {
>  		/*
>  		 * Send out the current raft state of the instance. Don't do
> -		 * that if Raft is disabled. It can be that a part of the
> -		 * cluster still contains old versions, which can't handle Raft
> -		 * messages. So when it is disabled, its network footprint
> -		 * should be 0.
> +		 * that if the remote instance is old. It can be that a part of
> +		 * the cluster still contains old versions, which can't handle
> +		 * Raft messages. Raft's network footprint should be 0 as seen
> +		 * by such instances.
>  		 */

Serge, why can't we send raft state for anon replicas as well? As far
as I understand anon replicas do receive raft updates from WAL notifications,
right? So this is somehow inconsistent that WAL's based raft updates are
reaching anon replicas while initial state from subscribe state is not,
or I miss something?