Просмотр исходного кода

Merge pull request #13964 from HJianBo/fix-batch-insert-in-dynamo

fix: the batch_query it not handle all msgs in a batch
JianBo He 1 год назад
Родитель
Сommit
6f3c03cf44

+ 1 - 1
apps/emqx_bridge_dynamo/src/emqx_bridge_dynamo.app.src

@@ -1,6 +1,6 @@
 {application, emqx_bridge_dynamo, [
     {description, "EMQX Enterprise Dynamo Bridge"},
-    {vsn, "0.2.3"},
+    {vsn, "0.2.4"},
     {registered, []},
     {applications, [
         kernel,

+ 2 - 2
apps/emqx_bridge_dynamo/src/emqx_bridge_dynamo_connector.erl

@@ -310,8 +310,8 @@ get_query_tuple([{_ChannelId, {_QueryType, _Data}} | _]) ->
         {unrecoverable_error,
             {invalid_request, <<"The only query type that supports batching is insert.">>}}
     );
-get_query_tuple([InsertQuery | _]) ->
-    get_query_tuple(InsertQuery).
+get_query_tuple([_InsertQuery | _] = Reqs) ->
+    lists:map(fun get_query_tuple/1, Reqs).
 
 ensuare_dynamo_keys({_, Data} = Query, State) when is_map(Data) ->
     ensuare_dynamo_keys([Query], State);

changes/ce/fix-13876.md → changes/ee/fix-13881.md


+ 1 - 0
changes/ee/fix-13964.md

@@ -0,0 +1 @@
+Fix the issue of data loss when enabling batch writing in DynamoDB Sink