-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add --use-decimals flag to TPC-DS ConvertFiles #1506
Add --use-decimals flag to TPC-DS ConvertFiles #1506
Conversation
Signed-off-by: Andy Grove <andygrove@nvidia.com>
084153b
to
528b53a
Compare
build |
integration_tests/src/main/scala/com/nvidia/spark/rapids/tests/tpcds/TpcdsLikeSpark.scala
Show resolved
Hide resolved
Signed-off-by: Andy Grove <andygrove@nvidia.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
conf.coalesce, | ||
conf.repartition, | ||
conf.withPartitioning()) | ||
baseInput = conf.input(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I think it's better to use named parameters consistently, so spark
should be spark = spark
, or is it too redundant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My main goal was to make sure the configuration options were being passed to the correct arguments, but it makes sense to use named args for all of them so I have updated this.
build |
build |
* Add --use-decimals flag to TPC-DS ConvertFiles Signed-off-by: Andy Grove <andygrove@nvidia.com> * update copyright Signed-off-by: Andy Grove <andygrove@nvidia.com> * make named args consistent
* Add --use-decimals flag to TPC-DS ConvertFiles Signed-off-by: Andy Grove <andygrove@nvidia.com> * update copyright Signed-off-by: Andy Grove <andygrove@nvidia.com> * make named args consistent
…VIDIA#1506) * Prevent optimization * Add comment Signed-off-by: Nghia Truong <nghiat@nvidia.com> * Revert "Add comment" This reverts commit 025fad50bf317b8608e9f41ea637d3131181eadc. --------- Signed-off-by: Nghia Truong <nghiat@nvidia.com>
This adds the ability to specify to use decimal types when converting TPC-DS data to Parquet when using spark-submit with the ConvertFiles utility,