Skip to content

Commit

Permalink
Merge pull request #345 from brooklyn-data/fix/add-wide-number-mode-t…
Browse files Browse the repository at this point in the history
…o-bigquery-all-results-parsed-json

[Fix] Add wide_number_mode to BigQuery all results columns
  • Loading branch information
jared-rimmer authored May 25, 2023
2 parents b7d45ac + f32b6e2 commit 6f69b46
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion macros/upload_exposures.sql
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
'{{ exposure.package_name }}', {# package_name #}
{{ tojson(exposure.depends_on.nodes) }}, {# depends_on_nodes #}
{{ tojson(exposure.tags) }}, {# tags #}
parse_json('{{ tojson(exposure) | replace("\\", "\\\\") | replace("'", "\\'") | replace('"', '\\"') }}') {# all_results #}
parse_json('{{ tojson(exposure) | replace("\\", "\\\\") | replace("'", "\\'") | replace('"', '\\"') }}', wide_number_mode=>'round') {# all_results #}
)
{%- if not loop.last %},{%- endif %}
{%- endfor %}
Expand Down
2 changes: 1 addition & 1 deletion macros/upload_models.sql
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
{{ tojson(model.tags) }}, {# tags #}
parse_json('''{{ tojson(model.config.meta) }}'''), {# meta #}
'{{ model.alias }}', {# alias #}
parse_json('{{ tojson(model) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}') {# all_results #}
parse_json('{{ tojson(model) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}', wide_number_mode=>'round') {# all_results #}
)
{%- if not loop.last %},{%- endif %}
{%- endfor %}
Expand Down
2 changes: 1 addition & 1 deletion macros/upload_seeds.sql
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@
'{{ seed.checksum.checksum }}', {# checksum #}
parse_json('''{{ tojson(seed.config.meta) }}'''), {# meta #}
'{{ seed.alias }}', {# alias #}
parse_json('{{ tojson(seed) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}') {# all_results #}
parse_json('{{ tojson(seed) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}', wide_number_mode=>'round') {# all_results #}
)
{%- if not loop.last %},{%- endif %}
{%- endfor %}
Expand Down
2 changes: 1 addition & 1 deletion macros/upload_snapshots.sql
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
'{{ snapshot.config.strategy }}', {# strategy #}
parse_json('''{{ tojson(snapshot.config.meta) }}'''), {# meta #}
'{{ snapshot.alias }}', {# alias #}
parse_json('{{ tojson(snapshot) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}') {# all_results #}
parse_json('{{ tojson(snapshot) | replace("\\", "\\\\") | replace("'","\\'") | replace('"', '\\"') }}', wide_number_mode=>'round') {# all_results #}
)
{%- if not loop.last %},{%- endif %}
{%- endfor %}
Expand Down

0 comments on commit 6f69b46

Please sign in to comment.