Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Temporary fix for breaking change in staging database #90

Merged
merged 2 commits into from
Mar 11, 2024

Conversation

harisang
Copy link
Contributor

There was a recent staging database change that renamed a column in the fee policies table.
cowprotocol/services#2470

This caused the dune-sycn job to start crashing with the following error:

psycopg2.errors.UndefinedColumn: column fp.max_volume_factor does not exist
LINE 93:                     fp.max_volume_factor * os.sell_amount * ...

This PR adds a temporary fix, which should be cleaned up this Tuesday as the change moves to prod as well with the upcoming weekly release.

@harisang harisang changed the title fix breaking change in staging database Temporary fix for breaking change in staging database Mar 10, 2024
@harisang harisang requested review from fhenneke and sunce86 March 10, 2024 14:08
@harisang harisang merged commit e19bc5e into main Mar 11, 2024
6 checks passed
@harisang harisang deleted the staging_column_renaming branch March 11, 2024 09:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants