Use IEC standard abbreviations (GiB, TiB, etc) #6742
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
resolves #6741
equivalent to dbt-labs/dbt-bigquery#482
Description
When using multiples of 1024, the IEC standard abbreviations are "GiB", "TiB", "PiB" instead of "GB", "TB", "PB" (which are for multiples of 1000).
More info
Today I Learned (TIL),
format_bytes
(andformat_rows_number
) are defined in dbt-core but redefined in dbt-bigquery and only used in that adapter. So the dbt-core version is unused to best I can tell.Context for the duplication
Removing them from dbt-core was mentioned here and here as part of the discussion for #48.
It feels to me like one of the following should have happened along with dbt-labs/dbt-bigquery#48:
format_bytes
andformat_rows_number
from dbt-core; orAs it stands, the implementations in dbt-core are unused (to the best that we can tell), but would probably need to be maintained to preserve backwards compatibility in case some unknown adapter is using them.
Proposal
Going forward, I'd propose that we choose one (and only one) of the following:
Checklist
changie new
to create a changelog entry