Skip to content

Commit 22eb1c3

Browse files
Update tests/test_groupby.py comment
Co-authored-by: Dimitri Yatsenko <[email protected]>
1 parent 66b7050 commit 22eb1c3

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

tests/test_groupby.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,9 @@
33

44
def test_aggr_with_proj():
55
# issue #944 - only breaks with MariaDB
6-
# Maria db wants all the fields in the select statement to be in the group by
7-
# even if the fields are in fuctional dependency, this is incorrect and MySQL
8-
# implements this properly
9-
# More info on the issue page
6+
# MariaDB implements the SQL:1992 standard that prohibits fields in the select statement that are
7+
# not also in the GROUP BY statement.
8+
# An improved specification in SQL:2003 allows fields that are functionally dependent on the group by
9+
# attributes to be allowed in the select. This behavior is implemented by MySQL correctly but not MariaDB yet.
10+
# See MariaDB issue: https://jira.mariadb.org/browse/MDEV-11588
1011
A.aggr(D.proj(m='id_l'), ..., n='max(m) - min(m)')

0 commit comments

Comments
 (0)