Opened 5 weeks ago

Last modified 7 days ago

#35487 new Cleanup/optimization

Removal of CASCADE in DROP COLUMN migration operation

Reported by: Petr Přikryl Owned by: nobody
Component: Migrations Version: dev
Severity: Normal Keywords:
Cc: Petr Přikryl, Adam Johnson Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Created based on discussion in ​https://forum.djangoproject.com/t/removefield-dropping-views-without-notice/31409

Arguments:

  • Consistency acrocss backends. Cascade is used for postgres backend only.
  • Cascade is no more needed as constraints and sequences are dropped explicitly. Next, sequences are being replaced by new identity columns.
  • No side effects - unexpected removals in DB schema.
  • Test suite is passing after cascade removal.

Change History (6)

comment:2 by Adam Johnson, 5 weeks ago

Cc: Adam Johnson added

comment:3 by Adam Johnson, 5 weeks ago

Has patch: set
Patch needs improvement: set
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → Cleanup/optimization
Version: 5.0 → dev

comment:4 by Petr Přikryl, 5 weeks ago

Patch needs improvement: unset

comment:5 by Sarah Boyce, 3 weeks ago

Needs tests: set
Patch needs improvement: set

Marking as "needs tests" as we either should write a test or have a discussion on why it's not required in this case 👍

comment:6 by Petr Přikryl, 7 days ago

Patch needs improvement: unset
Note: See TracTickets for help on using tickets.
Back to Top