[tarantool-patches] Re: [PATCH] Disable triggers for _collation during bootstrap

[Vladislav Shpilevoy <v.shpilevoy@tarantool.org>]

Hi! Thanks for the patch! LGTM.

On 15/11/2018 18:04, Nikita Pettik wrote:
> After patch that introduced "none" collation (a953051),
> box.internal.bootstrap() started to fail due to inability to drop
> mentioned collation. Lets turn off system triggers for _collation space
> in order to process its complete purification during bootstrap.
> ---
> Branch: https://github.com/tarantool/tarantool/tree/np/none-collation-hotfix
> 
>   src/box/bootstrap.snap  | Bin 1911 -> 1911 bytes
>   src/box/lua/upgrade.lua |   1 +
>   2 files changed, 1 insertion(+)
> 
> diff --git a/src/box/lua/upgrade.lua b/src/box/lua/upgrade.lua
> index 4b0fd0345..3d9acc976 100644
> --- a/src/box/lua/upgrade.lua
> +++ b/src/box/lua/upgrade.lua
> @@ -70,6 +70,7 @@ local function set_system_triggers(val)
>       box.space._func:run_triggers(val)
>       box.space._priv:run_triggers(val)
>       box.space._trigger:run_triggers(val)
> +    box.space._collation:run_triggers(val)
>   end
>   
>   --------------------------------------------------------------------------------
>