Minor changes according to review comments

Author: ita-sammann

crud/common/call.lua

@@ -19,13 +19,13 @@ local CRUD_CALL_FUNC_NAME = utils.get_storage_call(CALL_FUNC_NAME)
 
 -- Methods that can continue execution in fast mode when rebalance starts
 local safe_mode_allowed_fast_methods = {
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('readview_open_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('readview_close_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('select_readview_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('truncate_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('len_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('count_on_storage'),
-    CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('get_border_on_storage'),
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('readview_open_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('readview_close_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('select_readview_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('truncate_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('len_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('count_on_storage')] = true,
+    [CRUD_CALL_FUNC_NAME .. '/fast/' .. utils.get_storage_call('get_border_on_storage')] = true,
 }
 
 local call = {}
@@ -46,17 +46,8 @@ local function safe_mode_enable()
     call_on_storage = call_on_storage_safe
 
     for fb_id, fb in pairs(fiber.info()) do
-        if string.find(fb.name, CRUD_CALL_FUNC_NAME .. '/fast/') then
-            local do_kill = true
-            for _, allowed_method in ipairs(safe_mode_allowed_fast_methods) do
-                if fb.name == allowed_method then
-                    do_kill = false
-                    break
-                end
-            end
-            if do_kill then
-                fiber.kill(fb_id)
-            end
+        if fb.name:startswith(CRUD_CALL_FUNC_NAME .. '/fast/') and not safe_mode_allowed_fast_methods[fb.name] then
+            fiber.kill(fb_id)
         end
     end
 end

crud/common/rebalance.lua

@@ -31,6 +31,9 @@ local function safe_mode_trigger(_, new, space, op)
     if space ~= '_bucket' then
         return
     end
+    -- We are interested only in two operations that indicate the beginning of bucket migration:
+    --  * We are receiving a bucket (new bucket with status RECEIVING)
+    --  * We are sending a bucket to another node (existing bucket status changes to SENDING)
     if (op == 'INSERT' and new.status == vshard_consts.BUCKET.RECEIVING) or
             (op == 'REPLACE' and new.status == vshard_consts.BUCKET.SENDING) then
         box.broadcast(SAFE_MOD_ENABLE_EVENT, true)
@@ -60,6 +63,8 @@ end
 local function safe_mode_enable()
     if not box.info.ro then
         box.space[SETTINGS_SPACE_NAME]:replace{ 'safe_mode', true }
+        -- The trigger is needed to detect the beginning of rebalance process to enable safe mode.
+        -- If safe mode is enabled we don't need the trigger anymore.
         for _, trig in pairs(box.space._bucket:on_replace()) do
             if trig == safe_mode_trigger then
                 box.space._bucket:on_replace(nil, safe_mode_trigger)
@@ -78,6 +83,8 @@ end
 local function safe_mode_disable()
     if not box.info.ro then
         box.space[SETTINGS_SPACE_NAME]:replace{ 'safe_mode', false }
+        -- We have disabled safe mode so we need to add the trigger to detect the beginning
+        -- of rebalance process to enable safe mode again.
         box.space._bucket:on_replace(safe_mode_trigger)
     end
     M.safe_mode = false