[tarantool-patches] Re: [PATCH 2/2] Complete module reload

[Vladislav Shpilevoy]

>>> +M.discovery_f = discovery_f
>>> +M.failover_f = failover_f
>>
>> 9. Why did you move these lines?
>>
> Because M table should not be changed in case of
> `router_cfg(M.current_cfg)`
> failure

But now router_cfg on reload starts old M.failover_f and
old M.discovery_f, it is not? I think, that if cfg fails on
reload, then the caller should just return to the old
version.