-
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
DecimalType support for Aggregate Count #1476
Conversation
Turned it on and added test Signed-off-by: Raza Jafri <rjafri@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. I was expecting a change to an existing test rather than a new test, but I couldn't find a test that just did a count without combining it with other aggregates and/or filters.
build |
@@ -418,3 +418,10 @@ def test_arithmetic_reductions(data_gen): | |||
'avg(a)'), | |||
conf = _no_nans_float_conf) | |||
|
|||
@ignore_order | |||
@pytest.mark.parametrize('data_gen', all_gen, ids=idfn) | |||
@pytest.mark.parametrize('count_func', [f.count, f.countDistinct], ids=idfn) |
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.
Using a function in here with idfn is producing different names each time the test runs and is causing the parallel runs of the tests to fail. If you drop the ids=idfn
it fixes the problem
idfn causes a problem when passing a function as a parameter to the test Signed-off-by: Raza Jafri <rjafri@nvidia.com>
build |
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
…IDIA#1476) Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Turned on the Decimal Type for Count on and added test
Signed-off-by: Raza Jafri rjafri@nvidia.com
This fixes #1334