Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert 0x changes: downstream is broken #7189

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Revert 0x changes: downstream is broken #7189

merged 3 commits into from
Nov 25, 2024

Conversation

jeff-dude
Copy link
Member

@RantumBits unfortunately everything blew up in prod with all these changes merged :(
it's too difficult to pinpoint exactly what at the moment. we will need to revert and get prod running again, then revisit how we roll these changes out. we should always start as upstream in the pipeline as possible, ensure all is good, and continue from there

@jeff-dude jeff-dude added hotfix dbt: dex covers the DEX dbt subproject labels Nov 25, 2024
@jeff-dude jeff-dude marked this pull request as ready for review November 25, 2024 21:22
@jeff-dude jeff-dude merged commit c475f44 into main Nov 25, 2024
1 of 2 checks passed
@jeff-dude jeff-dude deleted the revert-0x branch November 25, 2024 21:22
@github-actions github-actions bot locked and limited conversation to collaborators Nov 25, 2024
@jeff-dude
Copy link
Member Author

@RantumBits even after the reverts to original state prior to today, when i tried to run a full refresh of the 0x spells, they were failing. example here:

23:17:16    Database Error in model zeroex_polygon_nft_fills (models/_projects/zeroex/polygon/zeroex_polygon_nft_fills.sql)
  TrinoUserError(type=USER_ERROR, name=TYPE_MISMATCH, message="line 42:28: Cannot apply operator: timestamp(3) with time zone <= varchar(10)", query_id=20241125_231241_02189_cv74k)

i'm not sure why a model that wasn't touched would do this. i assume it hasn't had a full refresh in a long time, so this error had been hidden or something?

to get prod running again, i had to exclude all spells in models/_projects/zeroex/ directory. we need to figure out how to get these running again. feel free to dm me on TG if you want to work through this and re-enable the spells

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dbt: dex covers the DEX dbt subproject hotfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant