way to throw errors between governor and the initiator

User story

1. Вызывается какой-либо SQL над плагинами на каком-то узле - CREATE PLUGIN, например
2. Инициируется асинхронный вызов говернора через _pico_property
3. Говернор получает ошибку при установке плагина, откатывает операцию и формирует сообщение об ошибке
4. Пользователь, вызвавший SQL должен увидеть сообщение об ошибке

changed milestone to %24.6 - Plugins and Tiers

added 1 deleted label

assigned to @vifley

Вижу 2 подхода:

1. Глобальная таблица request_id -> result:

• с вызовом какой-то операции через губера клиент должен будет отправлять также cas в эту таблицу
• после этого клиент сразу начинает ждать появления в этой таблице строчки
• губер выполнив запрос делает cas в эту таблицу
• увидев строчку он посылает cas на её удаление и возвращает пользователю соответствующий ответ
• Плюсы:
  • общий подход
• Минусы:
  • нагрузка на рафт
  • значительная задержка между моментом, когда результат известен и когда пользователь его увидит
  • нужно подчищать таблицу от устаревшей инфы, если клиенты таймаутятся

2. Добавлять request_id + requestor_id: InstanceId в запросы к губеру:

• для каждой операции через губера добавляем доп. поле с инфой об отправителе запроса
• модифицируем губера так, чтобы он отправлял RPC .proc_report_request_result на requestor_id, когда результат известен
• клиент отправив запрос ждёт, когда ему придёт RPC .proc_report_request_result, через какой-нибудь DistributedRequestManager::wait_request_result(request_id)
• Плюсы:
  • меньше задержка
  • нагрузка на рафт не меняется
  • общий подход (не смог придумать почему это не так)
• Минусы:
  • возможно реализация будет чуть сложнее, хотя не факт

Кажется надо идти по 2му пути

changed milestone to %Picodata next

added triaged + 1 deleted label

changed milestone to %24.6 - Plugins and Tiers

changed title from RFC: describe the way to throw errors between governor and the initiator to Операции управления плагинами должны возвращать человеко-читаемую диагностику при ошибке

changed the description

marked this issue as related to #813 (closed)

changed the description

changed title from Операции управления плагинами должны возвращать человеко-читаемую диагностику при ошибке to Операции управления плагинами должны возвращать человеко-читаемую ошибку

marked this issue as related to #586

После обсуждения пришли к выводу, что надо делать через добавление сообщения об ошибке и (возможно) айди запроса в Abort запись в рафт журнале. Для тех операций, у которых такая запись ещё не добавляется, нужно добавить эту запись

assigned to @gmoshkin and unassigned @vifley

removed the relation with #813 (closed)

mentioned in issue #586

added 1 deleted label

changed title from Операции управления плагинами должны возвращать человеко-читаемую ошибку to way to throw errors between governor and the initiator

mentioned in issue tarantool-module#221

marked this issue as related to tarantool-module#221

mentioned in merge request !1218 (merged)

closed with merge request !1218 (merged)

changed milestone to %24.5 - Lowercase