[Django] #28046: Add tablespace support to Index class

12 views
Skip to first unread message

Django

unread,
Apr 7, 2017, 4:07:33 AM4/7/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus | Owner: nobody
Holtermann |
Type: New | Status: new
feature |
Component: Database | Version: master
layer (models, ORM) |
Severity: Normal | Keywords:
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
For now, the `django.db.models.indexes.Index` class determines a non-
default tablespace from its model field if the index is defined on a
single field (`Field.db_tablespace`) or from a model's
`_meta.db_tablespace` for multi-column indexes. Indexes on expressions,
however, won't be able to identify which columns they refer to.

The initial API design would be to add `db_tablespace` or `tablespace` as
an attribute to the `Index` class and use that if provided, and fallback
to the current behavior if not given.

--
Ticket URL: <https://code.djangoproject.com/ticket/28046>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Apr 7, 2017, 10:21:27 AM4/7/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:1>

Django

unread,
Apr 21, 2017, 4:50:06 AM4/21/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: assigned

Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* owner: nobody => felixxm
* status: new => assigned


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:2>

Django

unread,
Apr 21, 2017, 4:28:06 PM4/21/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Markus Holtermann):

* has_patch: 0 => 1
* needs_tests: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:3>

Django

unread,
Apr 26, 2017, 4:46:24 PM4/26/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* needs_tests: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:4>

Django

unread,
Jun 20, 2017, 2:54:40 PM6/20/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* needs_better_patch: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:5>

Django

unread,
Jun 27, 2017, 2:43:48 PM6/27/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* needs_better_patch: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:6>

Django

unread,
Jun 27, 2017, 3:15:29 PM6/27/17
to django-...@googlegroups.com
#28046: Add tablespace support to Index class
-------------------------------------+-------------------------------------
Reporter: Markus Holtermann | Owner: felixxm
Type: New feature | Status: closed

Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by GitHub <noreply@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"3297dede7fce4b190f7b3bf0b0fc29a734151b61" 3297dede]:
{{{
#!CommitTicketReference repository=""
revision="3297dede7fce4b190f7b3bf0b0fc29a734151b61"
Fixed #28046 -- Added the db_tablespace parameter to class-based indexes.

Thanks Markus Holtermann and Tim Graham for reviews.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/28046#comment:7>

Reply all
Reply to author
Forward
0 new messages