[Django] #21652: responsive output of ``loaddata`` about how many objects got loaded

22 views
Skip to first unread message

Django

unread,
Dec 22, 2013, 3:03:49 PM12/22/13
to django-...@googlegroups.com
#21652: responsive output of ``loaddata`` about how many objects got loaded
--------------------------------------------+--------------------
Reporter: while0pass | Owner: nobody
Type: New feature | Status: new
Component: Core (Management commands) | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 1
Easy pickings: 0 | UI/UX: 0
--------------------------------------------+--------------------
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.

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

Django

unread,
Dec 22, 2013, 3:32:42 PM12/22/13
to django-...@googlegroups.com
#21652: responsive output of ``loaddata`` about how many objects got loaded
-------------------------------------+-------------------------------------

Reporter: while0pass | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 1
Has patch: 1 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by charettes):

* needs_docs: => 1
* needs_better_patch: => 0
* needs_tests: => 0
* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:1>

Django

unread,
Dec 22, 2013, 4:23:44 PM12/22/13
to django-...@googlegroups.com
#21652: responsive output of ``loaddata`` about how many objects got loaded
-------------------------------------+-------------------------------------

Reporter: while0pass | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 1
Has patch: 1 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by while0pass):

* cc: while0pass@… (added)


--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:2>

Django

unread,
Dec 24, 2013, 1:31:44 PM12/24/13
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------

Reporter: while0pass | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 1
Has patch: 1 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------

--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:3>

Django

unread,
Apr 13, 2015, 6:18:39 PM4/13/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned

Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by yitzc):

* owner: nobody => yitzc
* needs_docs: 1 => 0
* status: new => assigned


Comment:

Converted attached patch to [https://github.com/django/django/pull/4498
PR]
Asking around in the sprint room people agreed that this doesn't really
need its own documentation...

--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:4>

Django

unread,
Apr 13, 2015, 6:19:11 PM4/13/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by yitzc):

* cc: yitzc (added)


--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:5>

Django

unread,
Apr 14, 2015, 2:15:36 PM4/14/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:6>

Django

unread,
Apr 14, 2015, 6:33:55 PM4/14/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by charettes):

* needs_better_patch: 0 => 1
* stage: Ready for checkin => Accepted


Comment:

I think using `\r` would make more sense here.

--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:7>

Django

unread,
Apr 16, 2015, 8:01:22 AM4/16/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* needs_better_patch: 1 => 0


Comment:

[https://github.com/django/django/pull/4520 Updated PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:8>

Django

unread,
Apr 16, 2015, 9:39:53 AM4/16/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by charettes):

* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:9>

Django

unread,
Apr 16, 2015, 9:44:17 AM4/16/15
to django-...@googlegroups.com
#21652: Make ``loaddata`` notify about how many objects from fixture got processed
-------------------------------------+-------------------------------------
Reporter: while0pass | Owner: yitzc
Type: New feature | Status: closed

Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham <timograham@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"188a2418807f661eedfc675731521f2f46a3c49a" 188a2418]:
{{{
#!CommitTicketReference repository=""
revision="188a2418807f661eedfc675731521f2f46a3c49a"
Fixed #21652 -- Added notification when processing objects in loaddata

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

--
Ticket URL: <https://code.djangoproject.com/ticket/21652#comment:10>

Reply all
Reply to author
Forward
0 new messages