@Korablev77 нужная пояснительная бригада... Разве on_schema_init

Question

@Korablev77 нужная пояснительная бригада... Разве on_schema_init

не должен ВСЕГДА вызываться до окончания поднятия снапшота (тоесть перед началом), или триггер вызывается несколько раз ?

https://github.com/tarantool/tarantool/commit/3bc4a156e937102f23e2157ef88aa6c007759005#diff-bdd09286e91e43faaa5d3069d0ca967e302b2b32af52d9e3083c3386e98b2860R35

#database #programming #russian #software #tarantool

0

15.01.2021

16 ответов

13 просмотров

Dmitry Sharonov

да, но как он бы об этом узнал?

0

15.01.2021

Dmitry Sharonov

а он не прилег, просто у него теперь в спейсе куча дубликатов

0

15.01.2021

R-omk Автор вопроса

ну а зачем он туда писал их ... я не пойму в чем проблема то , не хочешь дублей - не пиши их туда

0

15.01.2021

Dmitry Sharonov

еще раз, есть бизнес-триггер. он разгребает то что приезжает по репликации например и что-то довставляет. его навешивали в on_schema_init чтобы когда репликация поедет он уже был. теперь понятно?

0

15.01.2021

R-omk Автор вопроса

это я хорошо понимаю , дело в том что on_schema_init срабатывает (во всяком случае раньше) ДО того как данные начнут загружаться из снапшота (тоесть это сильно раньше чем в приницпе до репликации дело доходит), так вот в этот момент уже можно менять схему или таплы... тоесть is_recovery_finished() никак не может быть == TRUE поскольку дело еще даже не дошло до поднятия снапшота с диска и xlog ... а то что я вижу на приведенном скриншоте полностью рушит мое представление о мире

0

15.01.2021

R-omk Автор вопроса

https://www.tarantool.io/en/doc/latest/reference/reference_lua/box_ctl/#lua-function.box.ctl.on_schema_init

0

15.01.2021

R-omk Автор вопроса

кажется все это вообще не решает то что описано в 5 пункте https://github.com/tarantool/tarantool/issues/5304#issuecomment-694085235 тут либо вообще нужно запрещать изменять любые данные в спейсах отличных от текущего тапла либо индексы в temporary и local должны к этому времени существовать... is_recovery_finished никак не может помочь в этом вопросе.... —- как я понимаю изначально ктото пытался сделать такую штуку чтобы при поднятии из снапшота создавалось некое представление во временном спейсе, это довольно логично для различных оптимизаций (например материализатор счетчиков ), но индексов в спейсе нет и таплы просто вставлялись как есть вместо агрегации (суммирования в случае со счетчиками), @a_lyapunov, обрати внимание, твой тикет

0

15.01.2021

Aleksandr Lyapunov

я подозреваю, что это бага, но бага в тесте

0

15.01.2021

R-omk Автор вопроса

допустим, тест я вообще плохо понял... я описал пример с материализацией счетчиков , когда они при поднятии снапшота считаются, такое вообще возможно после такого коммита?

0

15.01.2021

Aleksandr Lyapunov

Емнип там всё просто. если хочешь считать счетчики в temp спейсе - то делать этого было невозможно ранее и невоможно сейчас. Но сейчас есть еще та самая функция, чтобы проверить - можно уже или еще нет. Если хочешь считать счетчики в lua - пожалуйста. это можно было и можно сейчас. ничего проверять не нужно.

0

15.01.2021

R-omk Автор вопроса

тоесть выходит что из триггеров все равно нельзя писать в другие спейсы (во время бустрапа), тогда твой пример непонятен, или имеется ввиду что после коммита пример просто будет валиться с ошибкой вместо того чтобы дубли создавать в спейсах... ... тогда и в тесте проверку нужно делать внутри on replace а не в on schema init

0

15.01.2021

Aleksandr Lyapunov

да, нельзя. да, ты прав, должна быть внутри on_replace. эта проблема только в тесте, я проверил.

0

15.01.2021

R-omk Автор вопроса

тогда нужен еще какой нибудь триггер который в момент is_recovery_finished происходит, и до запуска репликации

0

15.01.2021

Michael Filonenko

звучит норм