Merge pull request #15228 from edx/efischer/revert_temp_logging
Revert "Merge pull request #15221 from edx/efischer/celery_logging"
This commit is contained in:
@@ -48,7 +48,7 @@ class BaseInstructorTask(Task):
|
||||
This is JSON-serialized and stored in the task_output column of the InstructorTask entry.
|
||||
|
||||
"""
|
||||
TASK_LOG.info('Task %s: success returned with progress: %s', task_id, task_progress)
|
||||
TASK_LOG.debug('Task %s: success returned with progress: %s', task_id, task_progress)
|
||||
# We should be able to find the InstructorTask object to update
|
||||
# based on the task_id here, without having to dig into the
|
||||
# original args to the task. On the other hand, the entry_id
|
||||
@@ -81,7 +81,7 @@ class BaseInstructorTask(Task):
|
||||
Note that there is no way to record progress made within the task (e.g. attempted,
|
||||
succeeded, etc.) when such failures occur.
|
||||
"""
|
||||
TASK_LOG.info(u'Task %s: failure returned', task_id)
|
||||
TASK_LOG.debug(u'Task %s: failure returned', task_id)
|
||||
entry_id = args[0]
|
||||
try:
|
||||
entry = InstructorTask.objects.get(pk=entry_id)
|
||||
|
||||
@@ -210,11 +210,7 @@ class CourseGradeReport(object):
|
||||
batched_rows = self._batched_rows(context)
|
||||
|
||||
context.update_status(u'Compiling grades')
|
||||
try:
|
||||
success_rows, error_rows = self._compile(context, batched_rows)
|
||||
except Exception: # pylint:disable=broad-except
|
||||
TASK_LOG.exception('Something failed in compilation')
|
||||
return context.update_status('Failed grades')
|
||||
success_rows, error_rows = self._compile(context, batched_rows)
|
||||
|
||||
context.update_status(u'Uploading grades')
|
||||
self._upload(context, success_headers, success_rows, error_headers, error_rows)
|
||||
|
||||
Reference in New Issue
Block a user