Opened 9 years ago

Closed 9 years ago

#24301 closed New feature (fixed)

Add missing aggregate functions for contrib.postgres

Reported by: Andriy Sokolovskiy Owned by: Andriy Sokolovskiy
Component: contrib.postgres Version: dev
Severity: Normal Keywords:
Cc: sokandpal@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Since django has contrib.postgres package, it will be useful to add some postgres-specific aggregate functions, here is a list of them:
http://www.postgresql.org/docs/9.4/static/functions-aggregate.html

Change History (11)

comment:1 by Andriy Sokolovskiy, 9 years ago

Cc: sokandpal@… added
Owner: set to Andriy Sokolovskiy
Status: newassigned

comment:2 by Tim Graham, 9 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Andriy Sokolovskiy, 9 years ago

I'm working on ticket in this branch:
https://github.com/coldmind/django/compare/postresql_aggregate_functions

Some early feedback will be appreciated.

comment:4 by Simon Charette, 9 years ago

I think the best way to get early feedback would be to open a PR against Django master since the comments on the changes will be aggregated and persisted through rebases.

comment:6 by Andriy Sokolovskiy, 9 years ago

Has patch: set

Need to final review

comment:7 by Tim Graham, 9 years ago

Patch needs improvement: set

Reviewed PR.

comment:8 by Andriy Sokolovskiy, 9 years ago

Patch needs improvement: unset

Fixed comments in PR, also added release notes.

comment:9 by Tim Graham, 9 years ago

Patch needs improvement: set

Did one more round of review. Should be ready for checkin after that.

comment:10 by Andriy Sokolovskiy, 9 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Fixed last comments

comment:11 by Tim Graham <timograham@…>, 9 years ago

Resolution: fixed
Status: assignedclosed

In e4cf8c8:

Fixed #24301 -- Added PostgreSQL-specific aggregate functions

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