Opened 20 months ago

Closed 4 months ago

#21652 closed New feature (fixed)

Make ``loaddata`` notify about how many objects from fixture got processed

Reported by: while0pass Owned by: yitzc
Component: Core (Management commands) Version: master
Severity: Normal Keywords:
Cc: while0pass@…, yitzc Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

When fixture is not very small, it may take perceptibly a long while to load it. And currently there is no output from the command irrespective of verbosity level. It usually makes people feel anxious when script freezes for a long time without any output.

I have a basic patch for loaddata (see attached diff file), but I could try making something more informative if the idea is welcomed.

Attachments (1)

ticket.21652--r.046a8eb9c499e0281a8e263a66e05dd6305d9e4e.diff (1.1 KB) - added by while0pass 20 months ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 20 months ago by charettes

  • Needs documentation set
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 20 months ago by while0pass

  • Cc while0pass@… added

comment:3 Changed 19 months ago by while0pass

  • Summary changed from responsive output of ``loaddata`` about how many objects got loaded to Make ``loaddata`` notify about how many objects from fixture got processed

comment:4 Changed 4 months ago by yitzc

  • Needs documentation unset
  • Owner changed from nobody to yitzc
  • Status changed from new to assigned

Converted attached patch to PR
Asking around in the sprint room people agreed that this doesn't really need its own documentation...

comment:5 Changed 4 months ago by yitzc

  • Cc yitzc added

comment:6 Changed 4 months ago by timgraham

  • Triage Stage changed from Accepted to Ready for checkin

comment:7 Changed 4 months ago by charettes

  • Patch needs improvement set
  • Triage Stage changed from Ready for checkin to Accepted

I think using \r would make more sense here.

comment:8 Changed 4 months ago by timgraham

  • Patch needs improvement unset

comment:9 Changed 4 months ago by charettes

  • Triage Stage changed from Accepted to Ready for checkin

comment:10 Changed 4 months ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 188a2418:

Fixed #21652 -- Added notification when processing objects in loaddata

Added a running count of the objects processed by loaddata when
verbosity >= 3.

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