Skip to content

Conversation

@ginahagg
Copy link

@ginahagg ginahagg commented Feb 17, 2022

Only files that have changed are rig_persist.erl and rig_persist_utils.erl. The rest are there due to formatting.. So ignore them.
PS: we aren't going to commit back to open source since it is too specific to our use..

Although this is our fork, not going to list any private information like JIRA ticket etc.

FILES CHANGED:
rig.hrl (formatting)
rig_sup.erl (add rig_persist gen server start)
rebar3 (update to latest)

NEW FILES:
rig_persist.erl (gen_server subscribed to be called for update event)
rig_persist_utils.erl (helpers)

From sjc-delivery-4:

Feb 22 21:15:27 localhost 2022-02-23T02:15:27 sjc-delivery-4 rtb-gateway <0.117.0> sasl <0.520.0> persist_map: [#{country => ["US"],
deals => [{6,[<<"ADG10342SAM">>]}],
flight_id => 169433,
start_date => {{2020,10,6},{17,0,0}}},
#{country => ["DE"],
deals => [{6,[<<"ADG10288SAM">>]}],
flight_id => 211224,
start_date => {{2022,3,31},{4,0,0}}},
#{country => ["US"],
deals => [{6,[<<"ADG10342SAM">>]}],
flight_id => 296282,
start_date => {{2022,2,17},{5,0,0}}},
#{country => ["ES"],
deals => [{6,[<<"ADG10288SAM">>]}],
flight_id => 211223,
start_date => {{2021,3,5},{3,0,0}}},
#{country => ["KR"],
deals => [{6,[<<"ADG10344SAM">>]}],
flight_id => 248159,
start_date => {{2021,8,20},{8,0,0}}},
#{country => ["DE"],
deals => [{6,[<<"ADG10342SAM">>]}],
flight_id => 296697,
start_date => {{2022,2,23},{16,0,0}}},
#{country => ["CA"],
deals => [{6,[<<"ADG10342SAM">>]}],
flight_id => 287208,
start_date => {{2022,3,28},{4,0,0}}},
#{country => ["KR"],
deals => [{6,[<<"ADG10288SAM">>]}],
flight_id => 143204,
start_date => {{2020,5,29},{22,0,0}}},
#{country => [<<"US">>],
deals => [{6,[<<"ADG10288SAM">>]}],
flight_id => 240573,
start_date => {{2021,7,14},{21,0,0}}}

@ginahagg ginahagg requested a review from ckvattes February 22, 2022 23:59
@ginahagg ginahagg merged commit bb2a95d into master Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants