Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid flattening a queryset

Browse Source 
notify_credentials was flattening a queryset when it shouldn't have,
which was exploding a large query all into memory. Hopefully this will
help with that.
This commit is contained in:
Michael Terry
2018-07-05 15:46:44 -04:00
committed by Michael Terry
parent 5c41fc69e9
commit f98415377d
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
openedx/core/djangoapps/credentials/management/commands/notify_credentials.py
View File

@@ -145,8 +145,8 @@ class Command(BaseCommand):
# First, do certs
for i, cert in enumerate(certs, start=1):
log.info(
"Handling credential changes (%d of %d) for certificate %s",
i, len(certs), certstr(cert),
"Handling credential changes %d for certificate %s",
i, certstr(cert),
)
if delay:
time.sleep(delay)
@@ -165,8 +165,8 @@ class Command(BaseCommand):
# Then do grades
for i, grade in enumerate(grades, start=1):
log.info(
"Handling grade changes (%d of %d) for grade %s",
i, len(grades), gradestr(grade),
"Handling grade changes %d for grade %s",
i, gradestr(grade),
)
if delay:
time.sleep(delay)