DataJunction / dj

A metrics platform.
http://datajunction.io
MIT License
29 stars 13 forks source link

Query validation SQL shouldn't use own materialized tables #1010

Closed shangyian closed 1 month ago

shangyian commented 1 month ago

Summary

When building SQL for query validation, it should not use its own materialized tables (i.e., the availability state). This PR achieves that by making the SQL builder use the default build criteria that contains the target node name.

This also adds an additional feature to kick off a background task that rebuilds and updates the SQL in the queryrequest table after each SQL request, so that subsequen requests will get the most up-to-date SQL.

Test Plan

Deployment Plan

netlify[bot] commented 1 month ago

Deploy Preview for thriving-cassata-78ae72 canceled.

Name Link
Latest commit bbc862a7acd0b8c7cc76d86673501ed0e0414b0b
Latest deploy log https://app.netlify.com/sites/thriving-cassata-78ae72/deploys/664d911064ff170008507399