#25945: Referring to partially run squashed migrations
-------------------------------------+-------------------------------------
     Reporter:  jarekwg              |                    Owner:  jarekwg
         Type:  Bug                  |                   Status:  assigned
    Component:  Migrations           |                  Version:  1.9
     Severity:  Normal               |               Resolution:
     Keywords:  make squash          |             Triage Stage:  Accepted
  migrations partially run           |
  dependencies                       |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by jarekwg):

 * has_patch:  0 => 1


Old description:

> Consider the migration tree defined by:
> {{{
> my_app/migrations/
> 0001_initial.py
> 0002_blah.py -> dep on 0001_initial
> 0001_initial_squashed_0002_blah.py -> replaces 0001_initial and 0002_blah
> 0003_ohnoes.py -> dep on ????
> }}}
>
> If we try to make `0003` depend on `0001_initial_squashed_0002_blah`,
> then we get a NodeNotFoundError if the database is only up to
> `0001_initial`: it is currently in the 'unsquashed reality' in which
> `0001_initial_squashed_0002_blah` does not exist. Creating new databases
> from this setup does not raise any problems, nor does running the
> migrations on databases that aren't part-way through a squashed migration
> series since these will live in the 'squashed reality'.
>
> If we try to make `0003` depend on `0002_blah`, then things work fine in
> all cases.
>
> However `makemigrations` chooses to point newly created migrations at the
> squashed migration `0001_initial_squashed_0002_blah` rather than the last
> of the unsquashed ones `0002_blah`, causing the problem described above
> and making it unobvious that new migrations should still point at the old
> replaced migrations up until the point at which the squashed migrations
> get transitioned to normal migrations.
>
> Proposed solution:
>
> Fully support listing the dependency both ways, as
> `0001_initial_squashed_0002_blah` and as `0002_blah`, using fancy logic
> to ensure valid references to inapplicable migrations are appropriately
> rerouted.

New description:

 Consider the migration tree defined by:
 {{{
 my_app/migrations/
 0001_initial.py
 0002_blah.py -> dep on 0001_initial
 0001_initial_squashed_0002_blah.py -> replaces 0001_initial and 0002_blah
 0003_ohnoes.py -> dep on ????
 }}}

 If we try to make `0003` depend on `0001_initial_squashed_0002_blah`, then
 we get a NodeNotFoundError if the database is only up to `0001_initial`:
 it is currently in the 'unsquashed reality' in which
 `0001_initial_squashed_0002_blah` does not exist. Creating new databases
 from this setup does not raise any problems, nor does running the
 migrations on databases that aren't part-way through a squashed migration
 series since these will live in the 'squashed reality'.

 If we try to make `0003` depend on `0002_blah`, then things work fine in
 all cases.

 However `makemigrations` chooses to point newly created migrations at the
 squashed migration `0001_initial_squashed_0002_blah` rather than the last
 of the unsquashed ones `0002_blah`, causing the problem described above
 and making it unobvious that new migrations should still point at the old
 replaced migrations up until the point at which the squashed migrations
 get transitioned to normal migrations.

 Proposed solution:

 Fully support listing the dependency both ways, as
 `0001_initial_squashed_0002_blah` and as `0002_blah`, using fancy logic to
 ensure valid references to inapplicable migrations are appropriately
 rerouted.

 [https://github.com/django/django/compare/master...jarekwg:ticket_25945 /
 Patch]
 [https://github.com/django/django/pull/5939 / Pull Request]

--

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

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" 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].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.af416e728f71351f87c9fada9d3493e2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to