search

cowprotocol / dune-sync

Components for syncing off-chain data with Dune Community Sources
4 stars 1 forks source link

[order_rewards] Add surplus fee #17

Closed bh2smith closed 1 year ago

bh2smith commented 1 year ago

  1. This PR changes the schema of the order rewards table and requires coordination with the Dune team to update it on their end. Since the values are in WEI, to preserve accuracy we extract and store as string type.

  2. This PR also introduces a breaking change feature for "hard reset" which empties the AWS buckets and removes the same data from our local storage, before proceeding with the sync script.

This is one part of a bigger change to make limit order fees available in Dune Community sources (so that the slippage query can consume it for solver-rewards).

cc @nlordell

Tagging also @dsalv (since I have included your github handle in the README and wanted to make sure this is OK with you).

It is nice that we haven't already released this to production (because these kinds of changes should NOT be regular occurrences)

bh2smith commented 1 year ago

We will be following up this PR with another schema change that puts all but the order_uid and tx_hash into a single cell of type JSON.