Closed chelseatroy closed 2 months ago
sql.diff
sql.diff
sql.diff
sql.diff
⚠️ Only part of the diff is displayed.
sql.diff
sql.diff
@BenWu on monitoring
vs monitoring_derived
; happy to move it, want to first get a solid consensus on where this is supposed to go. @gkaberere thoughts?
^ Update on this: I can't tag George for some reason so I Slacked him!
sql.diff
@BenWu for now I'm leaving it in monitoring_derived
, but accepted your suggestions and also did some cleanup to remove the (now unused) CTE and group by column names!
sql.diff
@edugfilho I think the failure here is that same udf.sql::glam.histogram_cast_struct
test. I know you issued a fix for this; could you please let me know when it merges? I'll update once that's done 😺
sorry for this whole thing. @lelilia removed the failing test on https://github.com/mozilla/bigquery-etl/pull/5971 and I updated this branch, so it should pass now.
sql.diff
sql.diff
⚠️ Only part of the diff is displayed.
These are views for a dashboard that will allow decision-makers to track our compute costs on remotely run machine learning training jobs. The referenced tables have now been created in the bigquery console; after this merges, the next step is for me to expose these views for inclusion in Looker dashboards.
Checklist for reviewer:
<username>:<branch>
of the fork as parameter. The parameter will also show up in the logs of themanual-trigger-required-for-fork
CI task together with more detailed instructions.For modifications to schemas in restricted namespaces (see
CODEOWNERS
):┆Issue is synchronized with this Jira Task