diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2019-12-30 13:08:17 -0500 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2019-12-30 13:17:17 -0500 |
commit | 2d5fa22c7d53ff8109d47ba5ae4fe3b9849ddd09 (patch) | |
tree | b9d7df7d2b7058420468d76c0f61b38e94640afd /lib/sqlalchemy/dialects/postgresql/dml.py | |
parent | 9d4a58d35c53484a1de66396139fc34cd65f5be8 (diff) | |
download | sqlalchemy-2d5fa22c7d53ff8109d47ba5ae4fe3b9849ddd09.tar.gz |
Include GROUP BY in _should_nest_selectable criteria
Fixed bug where usage of joined eager loading would not properly wrap the
query inside of a subquery when :meth:`.Query.group_by` were used against
the query. When any kind of result-limiting approach is used, such as
DISTINCT, LIMIT, OFFSET, joined eager loading embeds the row-limited query
inside of a subquery so that the collection results are not impacted. For
some reason, the presence of GROUP BY was never included in this criterion,
even though it has a similar effect as using DISTINCT. Additionally, the
bug would prevent using GROUP BY at all for a joined eager load query for
most database platforms which forbid non-aggregated, non-grouped columns
from being in the query, as the additional columns for the joined eager
load would not be accepted by the database.
Fixes: #5065
Change-Id: I9a2ed8196f83297ec38012138d1a5acdf9e88155
Diffstat (limited to 'lib/sqlalchemy/dialects/postgresql/dml.py')
0 files changed, 0 insertions, 0 deletions