Rewrite the description of box.cfg.replication

[vlalykin]

the documentation is incorrect: https://www.tarantool.io/ru/doc/latest/reference/configuration/#cfg-replication-replication

en:

Since version 1.7.4. If replication is not an empty string, the instance is considered to be a Tarantool replica. The replica will try to connect to the master specified in replication with a URI (Universal Resource Identifier)

ru:

Для версий от 1.7.4. и выше. Если replication не содержит пустую строку, экземпляр считается репликой. Реплика попытается подключиться к мастеру, указанному в параметре replication по URI (унифицированному идентификатору ресурса)

this is wrong.

en:
however, in fact, in the cluster, both on the master and on the replica, the value 'replication' does not contain an empty string (it contains a list of all members of the replica-set)

ru:
на самом деле, в кластере, как на мастере, так и на реплике, значение 'replication' не содержит пустую строку (там содержится список всех членов replica-set)

[rindblack15]

After talking with Sergey P., it was decided to close the ticket.
"значение 'replication' не содержит пустую строку (там содержится список всех членов replica-set)" - the statement is incorrect, "box.cfg.replication" may not contain all replica-set members, a common situation.

[vlalykin]

текущая формулировка "Если replication не содержит пустую строку, экземпляр считается репликой" не верна. А именно: мастер содержит непустую строку (простейший кластер). По словам Alexander.Turenko (из команды mail.ru), В качестве критерия, реплика это или нет, должно использоваться box.cfg.read_only

Привожу переписку:

Victor.Lalykin: так все таки, на мастере box.cfg.replication nil или не nil ???
Alexander.Turenko: Обычно box.cfg.replication везде одинаковый, а у мастера ставится флаг box.cfg{read_only = true/false}.

[veod32]

@sergepetrenko @Totktonada Сергей, Саша, можете прокомментировать по переписке выше?

[sergepetrenko]

@veod32

Короче, этот кусок в документации написан плохо. И то, как было, не до конца точно, и то, как предлагается.
Обычно box.cfg.replication == {список uri всех нод в репликасете}
Но никто не запрещает сделать box.cfg.replication пустым.

Говорить, что мастер - тот, у кого пустой box.cfg.replication не до конца правильно. Это действительно так (нода с пустым box.cfg.replication будет считать себя мастером), но, как мы договорились выше, чаще всего box.cfg.replication ни у кого не пустой. Тогда (по умолчанию, если box.cfg.read_only или, например, box.cfg.election_mode не трогать ) будет мультимастер. Каждая нода будет writeable и каждая нода будет мастером для остальных.

Я думаю, в доку стоит добавить пример настройки:
На всех box.cfg.replication = {список uri всех нод в кластере} , и при этом на мастере box.cfg.read_only = false, а на репликах box.cfg.read_only = true.
Надо ещё упомянуть, что указывать всех в box.cfg.replication рекомендуется.

Для контраста ещё стоит показать, что конфиг когда все реплики подключены звездой к одному мастеру (то есть у них в box.cfg.replication стоит master_uri, а у мастера - пустая строка) тоже возможен.

[vlalykin]

Чем больше примеров, тем лучше.
Но цель поднятия этого вопроса - это четкий критерий мастер/реплика. Такой критерий очень нужен.
Например, роль в момент отработки изменения конфигурации должна понять, запускать (или завершать) задачу или нет. Вот, скажем, событие (метод) stop в роли не получает никаких вх. параметров, а надо знать, мастер это или реплика.

[veod32]

Накину еще в эту тему: у нас есть большая глава в документации, посвященная репликации
https://www.tarantool.io/en/doc/latest/book/replication/

В частности, раздел про настройку, с объяcнениями и примерами конфигурации
https://www.tarantool.io/en/doc/latest/book/replication/repl_bootstrap/

Там есть такие комментарии к параметрам:

• the box.cfg() replication parameter defines the URIs at which all instances in the replica set can accept connections. It includes the replica’s URI as well, although the replica is not a replication source right now. This parameter is mandatory only for master-master or full-mesh cluster setups.

• the read_only = false parameter setting enables data-change operations on the instance and makes the instance act as a master, not as a replica. That is the only parameter setting in our instance files that will differ.

[patiencedaur]

Will be done in the scope of Replication tech debt.