Ticket to add support for introspection of index type - https://code.djangoproject.com/ticket/27097
On Friday, 19 August 2016 21:03:00 UTC+5:30, Tim Graham wrote: > > I think if we do a similar thing like we do for field introspection > (outputting a comment like "# This field type is a guess" next to unknown > fields) it would be okay. The comment might say the backend doesn't support > index type introspection. Let's put out a call for help to try to get index > introspection done on the other databases for 1.11. > > On Wednesday, August 17, 2016 at 12:01:51 PM UTC-4, akki wrote: >> >> Hi >> >> Currently the inspectdb command doesn't take the database indexes into >> account (Ticket - #27060 >> <https://code.djangoproject.com/ticket/27060#ticket>). While working on >> adding this feature, there was some confusion related to the inclusion of >> the index type. >> >> >> I have implemented this feature here >> <https://github.com/django/django/pull/7083> but since Django doesn't >> support introspection of the type of index, currently the generated models >> have the default (B-tree) index no matter what the type of the actual index >> in the database may be. So I would like seek some advice on whether it >> would be a good idea to move ahead with this approach with a comment at the >> beginning of the generated file with the other comments >> <https://github.com/django/django/blob/19e20a2a3f763388bba9263d56db34012e90cf5b/django/core/management/commands/inspectdb.py#L48> >> and >> integrate this basic implementation into Django (with an update on the >> ticket and keeping the ticket open, maybe?). >> >> Or should this be kept on a hold until Django supports introspection of >> the index type for it's backends (PostgreSQL has this feature with some >> concerns >> about the design >> <https://github.com/django/django/pull/7046#discussion-diff-74431956>) >> and be done correctly all at once. Once that happens we would either use >> the correct index type if it is supported by Django, else add a comment >> informing that the index type is not a match and should be created manually. >> >> >> Please let me know if any other clarification is required on this. Thanks >> for any help. >> >> Regards >> akki >> > -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/4f7e7d3a-77d2-4a11-aab1-568e1a0352b1%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
