fix: raise error when fail in public lua functions

Summarize the changes

Для публичных функций из core-router.lua в случае ошибки следует использовать error вместо return nil. Иначе lua.call_with_args не понимает, что вернулась ошибка и воспринимает nil как успех (который она не может прочитать с луа стека). В результате, вместо настоящей ошибки, пользователь видит ошибку чтения с луа стека.

Ensure that

• New code is covered by unit and integration tests.
• Related issues would be automatically closed with gitlab's closing pattern (Closes #1, #2).
• Public modules are documented (check the rendered version with cargo doc --open).
• (if PEST grammar is changed) EBNF grammar reflects these changes (check the result with railroad diagram generator.

Next steps

• Cherry-pick to: none
• Update sbroad submodule in picodata/picodata.
• (if EBNF grammar is changed) create a follow-up issue in picodata/docs.

sbroad-core/src/core-router.lua

@@ -308,7 +308,11 @@ local function multi_storage_dql(uuid_to_args, func, handler, opts, tier_name)
             f:discard()
         end
     end
-    return nil, err, err_uuid
+    local msg = "Unknown error"
+    if err ~= nil and err.message ~= nil then
+        msg = err.message
+    end
+    error(lerror.make("Error on replicaset " .. err_uuid .. ": " .. msg))
 end
 
 _G.group_buckets_by_replicasets = function(buckets, tier_name)
@@ -317,7 +321,7 @@ _G.group_buckets_by_replicasets = function(buckets, tier_name)
     for _, bucket_id in pairs(buckets) do
         local rs, err = router:route(bucket_id)
         if err ~= nil then
-            return nil, err
+            error(err)
         end
         local uuid = rs.uuid
         if map[uuid] then
@@ -336,7 +340,7 @@ _G.get_replicasets_from_buckets = function(buckets, tier_name)
     for _, bucket_id in pairs(buckets) do
         local rs, err = router:route(bucket_id)
         if err ~= nil then
-            return nil, err
+            error(err)
         end
         local uuid = rs.uuid
         if not map[uuid] then