[Django] #27097: Add introspection for index type

37 views
Skip to first unread message

Django

unread,
Aug 20, 2016, 5:44:44 AM8/20/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: master
(models, ORM) | Keywords: introspection db-
Severity: Normal | indexes
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Allow introspecting the type of index (e.g. btree, rtee). This is already
[https://github.com/django/django/blob/9f27735612fc775380d7801c68af0ea1c97cf5a3/django/db/backends/postgresql/introspection.py#L148
implemented for postgres] but is something that can be implemented for and
leveraged by all backends.

This would be useful for writing tests while implementing new index type
using the class based indexes. Also a requirement for it
[https://github.com/django/django/pull/7083#discussion-diff-74759691 was
felt] while [https://code.djangoproject.com/ticket/27060#ticket adding
indexes to inspectdb] ([https://groups.google.com/forum/#!topic/django-
developers/IKohkDzC4bE related discussion on mailing list]).

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

Django

unread,
Aug 20, 2016, 8:10:36 AM8/20/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

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

* keywords: introspection db-indexes => introspection db-indexes 1.11
* needs_better_patch: => 0
* needs_docs: => 0
* needs_tests: => 0
* stage: Unreviewed => Accepted


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

Django

unread,
Aug 25, 2016, 8:54:15 AM8/25/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: assigned

Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

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

* owner: nobody => akki
* status: new => assigned
* has_patch: 0 => 1


Comment:

PR: https://github.com/django/django/pull/7152

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

Django

unread,
Aug 26, 2016, 4:29:54 PM8/26/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1

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

* needs_better_patch: 0 => 1


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

Django

unread,
Aug 29, 2016, 12:49:03 PM8/29/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

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

* needs_better_patch: 1 => 0


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

Django

unread,
Aug 30, 2016, 8:50:28 AM8/30/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

In [changeset:"4c7bf83cde9f698cad019a9b808dbe45a832d9c3" 4c7bf83c]:
{{{
#!CommitTicketReference repository=""
revision="4c7bf83cde9f698cad019a9b808dbe45a832d9c3"
Refs #27097, #27098 -- Moved PostgreSQL index type introspection to
get_constraints().
}}}

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

Django

unread,
Aug 30, 2016, 9:45:24 AM8/30/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: assigned
Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution:
Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1

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

* needs_better_patch: 0 => 1


Comment:

Introspection doesn't seem to work on MySQL 5.5.

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

Django

unread,
Sep 2, 2016, 9:14:45 PM9/2/16
to django-...@googlegroups.com
#27097: Add introspection for index type
-------------------------------------+-------------------------------------
Reporter: akki | Owner: akki
Type: New feature | Status: closed

Component: Database layer | Version: master
(models, ORM) |
Severity: Normal | Resolution: fixed

Keywords: introspection db- | Triage Stage: Accepted
indexes 1.11 |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham <timograham@…>):

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


Comment:

In [changeset:"dbccf163b6e45bf2a673c249d4667360676acddc" dbccf16]:
{{{
#!CommitTicketReference repository=""
revision="dbccf163b6e45bf2a673c249d4667360676acddc"
Fixed #27097 -- Added index type introspection to built-in db backends.
}}}

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

Reply all
Reply to author
Forward
0 new messages