Replace fetchall by iteration
In #4949 (closed), one of the goal of the change was to replace .fetchall by iteration over the cursor.
Indeed for psycopg2, it will only safe the list creation in memory because psycopg2 fetchall the data any way (except if we use named cursor).
Any way, it is still good to iterate over the cursor instead of .fetchall if the resulted list is never stored.
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Cédric Krier added type::performance + 1 deleted label
added type::performance + 1 deleted label
- Cédric Krier added easy label
added easy label
- Cédric Krier assigned to @ced
assigned to @ced
- Cédric Krier added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Cédric Krier added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
review335831003 updated at https://codereview.tryton.org/335831003/#ps347931003
New changeset 25fa27914f88 by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account/rev/25fa27914f88- Roundup Robot added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- Roundup Robot closed
closed
New changeset 8c2c281a6531 by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account_deposit/rev/8c2c281a6531New changeset cce84836da4f by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account_dunning/rev/cce84836da4fNew changeset ea055cc5009b by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account_invoice/rev/ea055cc5009bNew changeset 13c9be5bfcaf by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account_payment_sepa/rev/13c9be5bfcafNew changeset 4fa6fd2e3dda by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/account_statement/rev/4fa6fd2e3ddaNew changeset 13e879e38c1c by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/analytic_account/rev/13e879e38c1cNew changeset 945ba1baf73e by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/commission/rev/945ba1baf73eNew changeset 95c963bbe219 by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/currency/rev/95c963bbe219New changeset 877a827c069c by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/marketing_automation/rev/877a827c069cNew changeset 508fd56d0ae5 by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/party/rev/508fd56d0ae5New changeset 20ebbd9e622b by Cédric Krier in branch 'default':
Replace cursor fetchall by iteration over cursor
https://hg.tryton.org/modules/project/rev/20ebbd9e622b