grouping with extra_selects produces invalid sql
|Reported by:||ikelly||Owned by:|
|Component:||Database layer (models, ORM)||Version:||master|
|Has patch:||no||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
From  on, I'm getting aggregation_regress test failures in Oracle, apparently because annotations with extra selects are adding the extra select aliases into the group by list. This produces invalid SQL (at least in Oracle), since you can't use column aliases defined at the same level within a group by clause.
The attached patch uses the extra select expression itself in the group by, rather than the alias. This passes the tests across all four included backends, and seems to work in general as long as the expression is not a scalar subquery.
Change History (3)
Changed 8 years ago by ikelly
comment:1 Changed 8 years ago by ikelly
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
- Resolution set to fixed
- Status changed from new to closed