Opened 16 years ago

Closed 2 years ago

#10621 closed New feature (duplicate)

Add a way to have an aggregate() result as a queryset

Reported by: Malcolm Tredinnick Owned by: nobody
Component: Database layer (models, ORM) Version:
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

If one is trying to construct a complex queryset, involving nesting other querysets, there's currently no way to include the results of an aggregate() call. For example, consider

class Item(models.Model):
   name = models.CharField(max_length=10)

If I want to include the number of Items in some other queryset, such as

Foo.objects.filter(items__in=[...]).annotate(num=Count("items").filter(num=item_count)

We can't write item_count (which would be Items.objects.aggregate(Count("id"))) as an inner queryset here, since the aggregate() call returns a dictionary, not a queryset.

The return value of aggregate() is fine, but this ticket is about adding a version that does return a queryset. Right now, there's no way to fake this with annotate(...).values(...) since that always introduces a GROUP BY clause and the whole point is not to group by anything.

Not sure whether adding a parameter to aggregate() is the right idea, or adding something to the annotate() route to specify "no grouping whatsoever". This is all 1.2 timeframe stuff, but it's something to think about.

Change History (8)

comment:1 by Jacob, 16 years ago

milestone: 1.2
Triage Stage: UnreviewedAccepted

comment:2 by Tobias McNulty, 15 years ago

Maybe I misunderstand the problem, but in (postgre)SQL there is no way to filter on an aggregate column, e.g.:

django_test=# SELECT "testapp_parent"."id", COUNT("testapp_child"."id") AS "num" 
FROM "testapp_parent" 
LEFT OUTER JOIN "testapp_child" ON ("testapp_parent"."id" = "testapp_child"."parent_id") 
WHERE "num" = 0 
GROUP BY "testapp_parent"."id";
ERROR:  column "num" does not exist
LINE 1: ...parent"."id" = "testapp_child"."parent_id") WHERE "num" = 0 ...

You can ORDER BY aggregate columns (so perhaps this idea is still useful) but I'm not sure it'll work with .filter(num=item_count)

comment:3 by Tobias McNulty, 15 years ago

Hmm, my statements above are patently false (you can do that with HAVING). Forget I ever said anything.

comment:4 by James Bennett, 15 years ago

milestone: 1.2

1.2 is feature-frozen, moving this feature request off the milestone.

comment:5 by Chris Beaven, 14 years ago

Severity: Normal
Type: New feature

comment:6 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:7 by Aymeric Augustin, 13 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:8 by Simon Charette, 2 years ago

Resolution: duplicate
Status: newclosed

#28296 is a duplicate which happens to have more up-to-date discussion and and associated patch so I'll close this one as a duplicate.

Note: See TracTickets for help on using tickets.
Back to Top