We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Describe the bug Calling the avg function with all null values should be null, not 0
avg
To Reproduce with datafusion-cli
datafusion-cli
❯ create table foo as values (null::bigint, 1); 0 rows in set. Query took 0.002 seconds. ❯ select avg(column1) from foo group by column2; +------------------+ | AVG(foo.column1) | +------------------+ | 0 | +------------------+
Expected behavior AVG of all nulls should be null
AVG
postgres=# create table foo as values (null::bigint, 1); SELECT 1 postgres=# select avg(column1) from foo group by column2; avg ----- (1 row)
Additional context
Found when upgrading IOx in https://github.com/influxdata/influxdb_iox/pull/6639
I think the issue is that there was a bug in the Row evaluator that was not being used until after #4924 was merged
I have a fix and will get it up shortly
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Describe the bug
Calling the
avg
function with all null values should be null, not 0To Reproduce
with
datafusion-cli
Expected behavior
AVG
of all nulls should be nullAdditional context
Found when upgrading IOx in https://github.com/influxdata/influxdb_iox/pull/6639
I think the issue is that there was a bug in the Row evaluator that was not being used until after #4924 was merged
I have a fix and will get it up shortly
The text was updated successfully, but these errors were encountered: