refactor(emqx): add 'extra' field to message record
This commit is contained in:
parent
5f3b146f15
commit
5274e0a726
|
@ -71,7 +71,9 @@
|
||||||
%% Message Payload
|
%% Message Payload
|
||||||
payload :: emqx_types:payload(),
|
payload :: emqx_types:payload(),
|
||||||
%% Timestamp (Unit: millisecond)
|
%% Timestamp (Unit: millisecond)
|
||||||
timestamp :: integer()
|
timestamp :: integer(),
|
||||||
|
%% not used so far, for future extension
|
||||||
|
extra = [] :: term()
|
||||||
}).
|
}).
|
||||||
|
|
||||||
-record(delivery, {
|
-record(delivery, {
|
||||||
|
|
|
@ -292,7 +292,8 @@ to_map(#message{
|
||||||
headers = Headers,
|
headers = Headers,
|
||||||
topic = Topic,
|
topic = Topic,
|
||||||
payload = Payload,
|
payload = Payload,
|
||||||
timestamp = Timestamp
|
timestamp = Timestamp,
|
||||||
|
extra = Extra
|
||||||
}) ->
|
}) ->
|
||||||
#{id => Id,
|
#{id => Id,
|
||||||
qos => QoS,
|
qos => QoS,
|
||||||
|
@ -301,7 +302,8 @@ to_map(#message{
|
||||||
headers => Headers,
|
headers => Headers,
|
||||||
topic => Topic,
|
topic => Topic,
|
||||||
payload => Payload,
|
payload => Payload,
|
||||||
timestamp => Timestamp
|
timestamp => Timestamp,
|
||||||
|
extra => Extra
|
||||||
}.
|
}.
|
||||||
|
|
||||||
%% @doc Message to tuple list
|
%% @doc Message to tuple list
|
||||||
|
|
|
@ -206,7 +206,9 @@ t_to_map(_) ->
|
||||||
{headers, #{}},
|
{headers, #{}},
|
||||||
{topic, <<"topic">>},
|
{topic, <<"topic">>},
|
||||||
{payload, <<"payload">>},
|
{payload, <<"payload">>},
|
||||||
{timestamp, emqx_message:timestamp(Msg)}],
|
{timestamp, emqx_message:timestamp(Msg)},
|
||||||
|
{extra, []}
|
||||||
|
],
|
||||||
?assertEqual(List, emqx_message:to_list(Msg)),
|
?assertEqual(List, emqx_message:to_list(Msg)),
|
||||||
?assertEqual(maps:from_list(List), emqx_message:to_map(Msg)).
|
?assertEqual(maps:from_list(List), emqx_message:to_map(Msg)).
|
||||||
|
|
||||||
|
@ -219,6 +221,8 @@ t_from_map(_) ->
|
||||||
headers => #{},
|
headers => #{},
|
||||||
topic => <<"topic">>,
|
topic => <<"topic">>,
|
||||||
payload => <<"payload">>,
|
payload => <<"payload">>,
|
||||||
timestamp => emqx_message:timestamp(Msg)},
|
timestamp => emqx_message:timestamp(Msg),
|
||||||
|
extra => []
|
||||||
|
},
|
||||||
?assertEqual(Map, emqx_message:to_map(Msg)),
|
?assertEqual(Map, emqx_message:to_map(Msg)),
|
||||||
?assertEqual(Msg, emqx_message:from_map(emqx_message:to_map(Msg))).
|
?assertEqual(Msg, emqx_message:from_map(emqx_message:to_map(Msg))).
|
||||||
|
|
Loading…
Reference in New Issue