Relax system check on fields intermediary tables for db_table collision when database routers are installed

166 views
Skip to first unread message

Anonymous Rabbit

unread,
Feb 13, 2020, 8:04:19 AM2/13/20
to Django developers (Contributions to Django itself)
Hello there,

This regards multi-database Django.

On https://github.com/django/django/pull/11630, a change was made to allow for multiple models with the same db_name, when a DATABASE_ROUTER is present. 
A ticket was open here for that issue: https://groups.google.com/forum/#!searchin/django-updates/29092%7Csort:date/django-updates/K1VuUaa9BUc/IDmjZoiRCQAJ . Eventually it gets marked with wontfix, but with a notice: "Feel free to reopen and offer a patch for evaluation if you think the implementation is straightforward."

I implemented the same changes of PR #11630, but with fields.E340 in mind (fields.E340: The field’s intermediary table <table name> clashes with the table name of <model>/<model>.<field name>.).

I would like to open a PR for it, but I'm not sure of the process I have to follow. Should the ticket be re-opened? Can I open the PR pointing at the wontfix ticket instead? Should I open a new ticket for it?

Best regards,
Xavier

Adam Johnson

unread,
Feb 13, 2020, 8:23:28 AM2/13/20
to django-d...@googlegroups.com
Please create a new ticket with a reference to the old one, and then a PR against that, since your change is for a different check.

--
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 django-develop...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/7819a6e1-3f47-41c8-a6c0-d35de3f37578%40googlegroups.com.


--
Adam

Anonymous Rabbit

unread,
Feb 13, 2020, 9:00:35 AM2/13/20
to django-d...@googlegroups.com
Before I move forward I just to want clarify, my changes are for the same check specified in ticket #29092. Should I still open a new ticket in lieu of re-opening #29092?

You received this message because you are subscribed to a topic in the Google Groups "Django developers (Contributions to Django itself)" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/django-developers/Ypks34xj1AY/unsubscribe.
To unsubscribe from this group and all its topics, send an email to django-develop...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/CAMyDDM3-F2%3DBC4bGqiMpf2cG5F47Hn217DgV3YaVGc%2BK9_VWAg%40mail.gmail.com.

Tim Graham

unread,
Feb 13, 2020, 12:13:07 PM2/13/20
to Django developers (Contributions to Django itself)
The change for that ticket is already released. Please open a new ticket.


On Thursday, February 13, 2020 at 9:00:35 AM UTC-5, Anonymous Rabbit wrote:
Before I move forward I just to want clarify, my changes are for the same check specified in ticket #29092. Should I still open a new ticket in lieu of re-opening #29092?

On Thu, 13 Feb 2020 at 13:23, Adam Johnson <m...@adamj.eu> wrote:
Please create a new ticket with a reference to the old one, and then a PR against that, since your change is for a different check.

On Thu, 13 Feb 2020 at 12:15, Anonymous Rabbit <xav...@zyper.com> wrote:
Hello there,

This regards multi-database Django.

On https://github.com/django/django/pull/11630, a change was made to allow for multiple models with the same db_name, when a DATABASE_ROUTER is present. 
A ticket was open here for that issue: https://groups.google.com/forum/#!searchin/django-updates/29092%7Csort:date/django-updates/K1VuUaa9BUc/IDmjZoiRCQAJ . Eventually it gets marked with wontfix, but with a notice: "Feel free to reopen and offer a patch for evaluation if you think the implementation is straightforward."

I implemented the same changes of PR #11630, but with fields.E340 in mind (fields.E340: The field’s intermediary table <table name> clashes with the table name of <model>/<model>.<field name>.).

I would like to open a PR for it, but I'm not sure of the process I have to follow. Should the ticket be re-opened? Can I open the PR pointing at the wontfix ticket instead? Should I open a new ticket for it?

Best regards,
Xavier

--
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 django-d...@googlegroups.com.


--
Adam

--
You received this message because you are subscribed to a topic in the Google Groups "Django developers (Contributions to Django itself)" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/django-developers/Ypks34xj1AY/unsubscribe.
To unsubscribe from this group and all its topics, send an email to django-d...@googlegroups.com.

Anonymous Rabbit

unread,
Feb 13, 2020, 6:33:18 PM2/13/20
to django-d...@googlegroups.com
Will do. Thanks.

To unsubscribe from this group and all its topics, send an email to django-develop...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/2e38cc16-ac9c-461b-80f8-8fc3e21d6a53%40googlegroups.com.
Reply all
Reply to author
Forward
0 new messages