Changes between Initial Version and Version 5 of Ticket #22620


Ignore:
Timestamp:
Aug 19, 2014, 5:57:23 PM (10 years ago)
Author:
Tim Graham
Comment:

For (1), I added a sentence about using DATABASE_ROUTERS to avoid any queries to the default database if you want to run without one.

For (2), I added a similar sentence about using routers to avoid creating tables where you don't want them, but avoided any specific recommendation as I think it's best to let users figure out what tables they want and where.

For (3), after further investigation, migrate does support the --database flag and as far as I could tell the documentation is accurate with respect to how it works.

Note that for apps with migrations sqlall is replaced by sqlmigrate. Given the complexity of migrations, it seems like a bad idea to recommend that as a "fine-grained" approach the way sqlall was, so I removed that suggestion.

Feedback on the patch is welcome.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #22620

    • Property Version 1.51.6
    • Property Triage Stage UnreviewedAccepted
    • Property Has patch set
  • Ticket #22620 – Description

    initial v5  
    44
    55So I did that and got the following:
    6 
     6{{{
    77"Internal Server Error: /admin/
    88Traceback (most recent call last):
     
    5858    raise ImproperlyConfigured("settings.DATABASES is improperly configured. "
    5959ImproperlyConfigured: settings.DATABASES is improperly configured. Please supply the ENGINE value. Check settings documentation for more details."
    60 
     60}}}
    6161I include the traceback just to show that nothing originated in code of mine. Now previously, and now in fact (I switched back), I have my project up and running fine with two databases and with the 'default' key value being the dictionary of parameters for the first database. So I have reason to suggest that perhaps the docs are wrong concerning the possibility of starting with 'default':{}. Maybe it's a legacy thing that is no longer viable.
    6262
Back to Top