Opened 10 years ago
Closed 10 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 , 10 years ago
Cc: | added |
---|---|
Owner: | set to |
Status: | new → assigned |
comment:2 by , 10 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:3 by , 10 years ago
comment:4 by , 10 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:8 by , 10 years ago
Patch needs improvement: | unset |
---|
Fixed comments in PR, also added release notes.
comment:9 by , 10 years ago
Patch needs improvement: | set |
---|
Did one more round of review. Should be ready for checkin after that.
comment:10 by , 10 years ago
Patch needs improvement: | unset |
---|---|
Triage Stage: | Accepted → Ready for checkin |
Fixed last comments
I'm working on ticket in this branch:
https://github.com/coldmind/django/compare/postresql_aggregate_functions
Some early feedback will be appreciated.