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

wrong results for count()/count(distinct) on the same attribute #2808

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

wrong results for count()/count(distinct) on the same attribute #2808

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

Comments

@monetdb-team
Copy link

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

Date: 2011-05-05 17:04:17 +0200
From: Daniel Boesswetter <<daniel.boesswetter>>
To: SQL devs <>
Version: 11.3.1 (Apr2011) [obsolete]
CC: @njnes

Last updated: 2011-05-31 13:59:29 +0200

Comment 15788

Date: 2011-05-05 17:04:17 +0200
From: Daniel Boesswetter <<daniel.boesswetter>>

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.17) Gecko/20110422 Ubuntu/9.10 (karmic) Firefox/3.6.17
Build Identifier:

When counting the number of values in a column and the number of distinct values in the same column in the same query, the result is wrong.

Reproducible: Always

Steps to Reproduce:

sql>select count(name) from tables;
+------+
| L17 |
+======+
| 37 |
+------+
1 tuple (3.446ms)
sql>select count(distinct name) from tables;
+------+
| L20 |
+======+
| 31 |
+------+
1 tuple (4.475ms)
sql>select count(name) , count(distinct name) from tables;
+------+------+
| L21 | L22 |
+======+======+
| 37 | 37 |
+------+------+

Actual Results:

37 / 37

Expected Results:

37 / 31

Comment 15793

Date: 2011-05-10 23:13:06 +0200
From: @njnes

Changeset aad21c635125 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

fix bug #2808 wrong results for count()/count(distinct) on the same attribute
(todo at test)

Comment 15795

Date: 2011-05-11 16:44:42 +0200
From: @njnes

Changeset 73af4e753888 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

fix bug #2808 wrong results for count()/count(distinct) on the same attribute
(todo at test)
(transplanted from aad21c6351253c566c3f63f0121141097ff2713b)

Comment 15796

Date: 2011-05-13 11:16:59 +0200
From: @njnes

Changeset c12f78cc0965 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

added test for bug #2808

Comment 15798

Date: 2011-05-13 15:28:18 +0200
From: @njnes

fixed by properly comparing aggr expressions (including flag indicating distinct)

Comment 15804

Date: 2011-05-13 16:01:23 +0200
From: @njnes

Changeset 4eb2498ec8e7 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

added test for bug #2808
(transplanted from c12f78cc0965b60bdffeda0f2bf1c853de1f232f)

Comment 15810

Date: 2011-05-16 21:50:58 +0200
From: @njnes

Changeset f169139660d8 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

add test for count-count-distinct bug #2808

check variable sized types on binary copy into (solves bug #2010).

Comment 15962

Date: 2011-07-17 19:50:46 +0200
From: @drstmane

Changeset 74e5863a3248 made by Stefan Manegold Stefan.Manegold@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

moved test count-count-distinct.Bug-2808 to be first in directory,
such that its output is independent of other tests
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