Skip to content
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

stddev_pop and stddev_samp break during a GROUP BY #3257

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

stddev_pop and stddev_samp break during a GROUP BY #3257

monetdb-team opened this issue Nov 30, 2020 · 0 comments
Labels

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2013-03-15 02:58:53 +0100
From: Anthony Damico <>
To: SQL devs <>
Version: 11.15.1 (Feb2013)
CC: ajdamico, @hannesmuehleisen

Last updated: 2013-04-29 14:51:27 +0200

Comment 18618

Date: 2013-03-15 02:58:53 +0100
From: Anthony Damico <>

User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/20100101 Firefox/19.0
Build Identifier:

sql>select stddev_pop(pppymt_ip) from b09;

sql>select stddev_samp(pppymt_ip) from b09;

But then include any GROUP BY and they break--

sql>select stddev_pop(pppymt_ip) from b09 group by metro;

sql>select stddev_samp(pppymt_ip) from b09 group by metro;

Reproducible: Always

Steps to Reproduce:

  1. run any select statement using stddev_samp and stddev_pop without aggregation
  2. run the same select statement, using GROUP BY

Actual Results:

TypeException:user.s6_1[9]:'aggr.substdev' undefined in: _17:bat[:any,:dbl] := a
ggr.substdev(_3:bat[:oid,:dbl], _14:bat[:oid,:oid], r1_14:bat[:oid,:oid], _1:bit
)
program contains errors

TypeException:user.s5_1[9]:'aggr.substdevp' undefined in: _17:bat[:any,:dbl] :=
aggr.substdevp(_3:bat[:oid,:dbl], _14:bat[:oid,:oid], r1_14:bat[:oid,:oid], _1:b
it)
program contains errors

Expected Results:

aggregated standard deviation calculations

Comment 18621

Date: 2013-03-15 14:48:54 +0100
From: @sjoerdmullender

Changeset 862c0ec8510d made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=862c0ec8510d

Changeset description:

aggr.stdev{,] and aggr.variance{,p} need "abort_on_error" argument.
This fixes bug #3257.

Comment 18622

Date: 2013-03-15 14:56:11 +0100
From: @sjoerdmullender

Changeset c2f138d54d2e made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=c2f138d54d2e

Changeset description:

Added test for bug #3257.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant