Fix a bug with updating existing materialization.

DataJunction / dj

A metrics platform.

http://datajunction.io

MIT License

35 stars 15 forks source link

Fix a bug with updating existing materialization. #1218

Closed agorajek closed 3 weeks ago

agorajek commented 3 weeks ago

Summary

When a materialization existed and was inactive, it was hard-to-impossible to recreate it because we were not cleaning up the old deactivated_at marker. Now we are.

Test Plan

In the UI.

[ ] PR has an associated issue: #
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

auto

netlify[bot] commented 3 weeks ago

Deploy Preview for thriving-cassata-78ae72 canceled.

Name	Link
Latest commit	b91915fba5a0bdd3f0e302c8a0ae752da9550f2b
Latest deploy log	https://app.netlify.com/sites/thriving-cassata-78ae72/deploys/6723feecfa59c20008860acf