[Fix] Add wide_number_mode to BigQuery all results columns - Githubissues

brooklyn-data / dbt_artifacts

A dbt package for modelling dbt metadata. https://brooklyn-data.github.io/dbt_artifacts

Apache License 2.0

331 stars 126 forks source link

[Fix] Add wide_number_mode to BigQuery all results columns #345

Closed jared-rimmer closed 1 year ago

jared-rimmer commented 1 year ago

Overview

In Release 2.4.0 we added the all_results object to models.
In some instances this contains a number that cannot be parsed to a string and results in the following error:

13:04:01  on-run-end failed, error:
13:04:01   Invalid input: Input number: 1685019801.1016695 cannot round-trip through string representation; error in PARSE_JSON expression

The BQ docs show the solution in the examples under parse_json
This PR adds it to the all_results columns where it wasn't set previously
I've opted to not put it on all objects as I believe at the moment this issue will only appear in the all_results column based on the objects that are being used

Update type - breaking / non-breaking

[X] Minor bug fix
[ ] Documentation improvements
[ ] Quality of Life improvements
[ ] New features (non-breaking change)
[ ] New features (breaking change)
[ ] Other (non-breaking change)
[ ] Other (breaking change)

What does this solve?

Outstanding questions

What databases have you tested with?

[ ] Snowflake
[X] Google BigQuery
[ ] Databricks
[ ] Spark
[ ] N/A