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

Select * on grouped view: wrong error "cannot use non GROUP BY column 'a1' in query results without an aggregate function" #7013

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Labels
bug normal SQL
Milestone

Comments

@monetdb-team
Copy link

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

Date: 2020-11-21 14:50:42 +0100
From: @swingbit
To: SQL devs <>
Version: 11.39.5 (Oct2020)
CC: @PedroTadim

Last updated: 2020-11-21 23:09:50 +0100

Comment 28280

Date: 2020-11-21 14:50:42 +0100
From: @swingbit

User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.198 Safari/537.36
Build Identifier:

CREATE VIEW v AS
SELECT a1, a2
FROM (VALUES (1,2)) as (a1,a2)
GROUP BY a1, a2;

SELECT a1 from v;
SELECT a2 from v;
SELECT a1, a2 from v;
SELECT * from v; -- this is equivalent to the previous one, but fails.

Reproducible: Always

$ mserver5 --version
MonetDB 5 server 11.39.8 (64-bit, 128-bit integers)
This is an unreleased version
Copyright (c) 1993 - July 2008 CWI
Copyright (c) August 2008 - 2020 MonetDB B.V., all rights reserved
Visit https://www.monetdb.org/ for further information
Found 12.0GiB available memory, 8 available cpu cores
Libraries:
libpcre: 8.44 2020-02-12
openssl: OpenSSL 1.1.1g FIPS 21 Apr 2020
libxml2: 2.9.10
Compiled by: @ee4e38e5556f (x86_64-pc-linux-gnu)
Compilation: /usr/bin/cc -DNDEBUG=1
Linking : /usr/bin/ld

Comment 28281

Date: 2020-11-21 23:07:05 +0100
From: MonetDB Mercurial Repository <>

Changeset 97cb2149ff53 made by Pedro Ferreira pedro.ferreira@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=97cb2149ff53

Changeset description:

Added test and fix for bug #7013, ie we can't innocently believe the dnode row list will match the actual number of rows of the values list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug normal SQL
Projects
None yet
Development

No branches or pull requests

3 participants