Opened 10 years ago

Last modified 10 years ago

#23996 new New feature

Allow AlterOrderWithRespectTo to order existing data

Reported by: Markus Holtermann Owned by:
Component: Migrations Version: dev
Severity: Normal Keywords:
Cc: Markus Holtermann Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Following #23983 it might be worth it to add a flag to AlterOrderWithRespectTo to order existing data (off by default for backwards compat and as this might cause serious performance problems on large tables). The MigrationQuestioner could then ask (and warn) for that flag.

Change History (7)

comment:1 by mhall1, 10 years ago

Owner: changed from nobody to mhall1
Status: newassigned

comment:2 by mhall1, 10 years ago

Triage Stage: UnreviewedAccepted

comment:3 by mhall1, 10 years ago

I apologize but I have to release this ticket because I'm leaving on vacation and its still waiting for approval by the core team.

comment:4 by mhall1, 10 years ago

Owner: mhall1 removed
Status: assignednew

comment:5 by Tim Graham, 10 years ago

Easy pickings: unset

@mhall1, you probably didn't realize since you marked the ticket as "Accepted" it doesn't appear in the unreviewed ticket queue to get feedback. Anyway, the ideas seems okay to me although I am not quite sure what the implementation would look like -- would the questioner iterate through all the rows and ask for an order # or something else?

comment:6 by Markus Holtermann, 10 years ago

I suggest to order the existing rows by the default ordering of the related model (the owrt is defined on).

comment:7 by mhall1, 10 years ago

Oh okay, thanks for the tip @timgraham.

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