Write parquet with decimal columns in load_test #217
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.
This enables using
load_testto create parquet files withdecimalcolumns as well as reading in parquet files withdecimalcolumns for running queries. However, as the csv reader/writer doesn't supportdecimalyet, thedecimalvalues have to be read in asstrand cast todecimalbefore they're written to parquet.Due to: rapidsai/cudf#8311 the
decimalcolumns appear to be typeobjecthowever, the underlying type isdecimaland they only appear asobjectwhen callingdf.dtypesdue to a metadata error.Due to: rapidsai/cudf#8354 the original precision of each decimal column is lost, but this shouldn't affect the underlying values.
CC: @randerzander