Opened 2 years ago

Closed 2 years ago

Last modified 5 months ago

#33201 closed Cleanup/optimization (wontfix)

RenameModel with db_table should be a noop.

Reported by: Iuri de Silvio Owned by: Iuri de Silvio
Component: Migrations Version: 3.2
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

A RenameModel operation that already has db_table defined must be a noop.

In Postgres, it drops and recreates foreign key constraints. In sqlite it recreates the table (as expected for a table renaming).

Change History (9)

comment:1 Changed 2 years ago by Iuri de Silvio

Owner: changed from nobody to Iuri de Silvio
Status: new → assigned

comment:2 Changed 2 years ago by Mariusz Felisiak

Has patch: set
Summary: RenameModel with db_table recreate constraints. → RenameModel with db_table should be a noop.
Triage Stage: Unreviewed → Accepted
Type: Bug → Cleanup/optimization

​PR

comment:3 Changed 2 years ago by Iuri de Silvio

I fixed the patch and it is waiting for review.

comment:4 Changed 2 years ago by Mariusz Felisiak

Triage Stage: Accepted → Ready for checkin

comment:5 Changed 2 years ago by Mariusz Felisiak <felisiak.mariusz@…>

Resolution: → fixed
Status: assigned → closed

In afeafd60:

Fixed #33201 -- Made RenameModel operation a noop for models with db_table.

comment:6 Changed 16 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In 166a3b32:

Fixed #33953 -- Reverted "Fixed #33201 -- Made RenameModel operation a noop for models with db_table."

Regression in afeafd6036616bac8263d762c1610f22241c0187.
This reverts afeafd6036616bac8263d762c1610f22241c0187.

Thanks Timothy Thomas for the report.

comment:7 Changed 16 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In 7d5ccbbe:

[4.1.x] Fixed #33953 -- Reverted "Fixed #33201 -- Made RenameModel operation a noop for models with db_table."

Regression in afeafd6036616bac8263d762c1610f22241c0187.
This reverts afeafd6036616bac8263d762c1610f22241c0187.

Thanks Timothy Thomas for the report.

Backport of 166a3b32632c141541d1c3f0eff18e1d8b389404 from main

comment:8 Changed 16 months ago by Mariusz Felisiak

Has patch: unset
Resolution: fixed → wontfix
Triage Stage: Ready for checkin → Unreviewed

comment:9 Changed 5 months ago by GitHub <noreply@…>

In 31090389:

Refs #33201 -- Avoided unnecessary queries when renaming models with db_table on SpatiaLite.

Note: See TracTickets for help on using tickets.
Back to Top