[Django] #25809: PostgreSQL 9.5 BRIN Index support in contrib.postgres
Django
----------------------------------+--------------------
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
----------------------------------+--------------------
pgsql 9.5 introduces BRIN index which will be nice to have in django
postgres.
--
Ticket URL: <https://code.djangoproject.com/ticket/25809>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by timgraham):
* needs_better_patch: => 0
* needs_tests: => 0
* needs_docs: => 0
Comment:
Do you have a proposed API?
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:1>
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by auvipy):
Replying to [comment:1 timgraham]:
> Do you have a proposed API?
not till now, but should be based on Class based Index Dep?
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:2>
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: | Triage Stage:
| Someday/Maybe
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by timgraham):
* stage: Unreviewed => Someday/Maybe
Comment:
I'd guess so.
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:3>
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by timgraham):
* stage: Someday/Maybe => Accepted
Comment:
This can likely move forward now that the framework for class-based
indexes is in master.
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:4>
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by auvipy):
Replying to [comment:4 timgraham]:
> This can likely move forward now that the framework for class-based
indexes is in master.
exactly. will this be part of gsoc?
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:5>
Django
Reporter: auvipy | Owner:
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 0 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by auvipy):
* keywords: => postgres, index, BRIN
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:6>
Django
Reporter: Asif Saifuddin Auvi | Owner: (none)
Type: New feature | Status: new
Component: contrib.postgres | Version: master
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
Changes (by Mads Jensen):
* needs_better_patch: 0 => 1
* has_patch: 0 => 1
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:7>
Django
Reporter: Asif Saifuddin Auvi | Owner: (none)
Component: contrib.postgres | Version: master
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mads Jensen):
* needs_better_patch: 1 => 0
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:8>
Django
Reporter: Asif Saifuddin Auvi | Owner: (none)
Component: contrib.postgres | Version: master
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mads Jensen):
* needs_better_patch: 0 => 1
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:9>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: assigned
Component: contrib.postgres | Version: master
Severity: Normal | Resolution:
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Changes (by Mads Jensen):
* owner: (none) => Mads Jensen
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:8>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: assigned
Severity: Normal | Resolution:
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham):
* needs_better_patch: 1 => 0
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:9>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: assigned
Severity: Normal | Resolution:
Keywords: postgres, index, | Triage Stage: Accepted
BRIN |
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mads Jensen):
* needs_better_patch: 1 => 0
Comment:
Updated according to review by Tim Graham. It's tricky finding a good
```sql_create_index``` that suits all backends, as the current design
requires. My solution just introduces a new one, and uses this in the
```as_sql```-method.
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:9>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: assigned
Severity: Normal | Resolution:
Keywords: postgres, index, | Triage Stage: Ready for
BRIN | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Markus Holtermann):
* stage: Accepted => Ready for checkin
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:10>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Component: contrib.postgres | Version: master
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mads Jensen <mje@…>):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"e585c43be91fb3e5005ddb4191f64142c62a2ec3" e585c43b]:
{{{
#!CommitTicketReference repository=""
revision="e585c43be91fb3e5005ddb4191f64142c62a2ec3"
Fixed #25809 -- Added BrinIndex support in django.contrib.postgres.
Thanks Tim Graham and Markus Holtermann for review.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:11>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Akshesh Doshi):
* keywords: postgres, index, BRIN => postgres, index, BRIN, db-indexes
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:12>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Markus Holtermann <info@…>):
In [changeset:"7e299c0e03a52052068db5108de97cf59174bfd9" 7e299c0]:
{{{
#!CommitTicketReference repository=""
revision="7e299c0e03a52052068db5108de97cf59174bfd9"
Refs #25809 -- Prefered imports from django.db.models.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:13>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"65e321b78161d83db408504290edbfbace267233" 65e321b7]:
{{{
#!CommitTicketReference repository=""
revision="65e321b78161d83db408504290edbfbace267233"
Refs #25809 -- Made a few late review comments for BrinIndex.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:14>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by GitHub <noreply@…>):
In [changeset:"d1d73480019fc3754b335878be07f00b5321a065" d1d7348]:
{{{
#!CommitTicketReference repository=""
revision="d1d73480019fc3754b335878be07f00b5321a065"
Refs #25809 -- Removed BrinIndex.__repr__().
In retrospect, implementing a __repr__() for index subclasses doesn't
seem worthwhile.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:15>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"fb42d0247136249ea81962474e9a6a2faf1755f1" fb42d024]:
{{{
#!CommitTicketReference repository=""
revision="fb42d0247136249ea81962474e9a6a2faf1755f1"
Refs #25809 -- Omitted pages_per_range from BrinIndex.deconstruct() if
it's None.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:16>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"c685b8f838f7f9411a2a65fba7e3893f15439e18" c685b8f8]:
{{{
#!CommitTicketReference repository=""
revision="c685b8f838f7f9411a2a65fba7e3893f15439e18"
[1.11.x] Refs #25809 -- Omitted pages_per_range from
BrinIndex.deconstruct() if it's None.
Backport of fb42d0247136249ea81962474e9a6a2faf1755f1 from master
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:17>
Django
Reporter: Asif Saifuddin Auvi | Owner: Mads
| Jensen
Type: New feature | Status: closed
Severity: Normal | Resolution: fixed
Keywords: postgres, index, | Triage Stage: Ready for
BRIN, db-indexes | checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"ff9543b351e79de78e7867f255ad15e51555ba4a" ff9543b]:
{{{
#!CommitTicketReference repository=""
revision="ff9543b351e79de78e7867f255ad15e51555ba4a"
Refs #25809, #28990 -- Added PostgreSQL version check for BrinIndex
support.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/25809#comment:18>