From 657249bd6d8f16ea4d15a35459e3c2eee9460b09 Mon Sep 17 00:00:00 2001 From: Qubad786 Date: Thu, 4 May 2017 13:00:43 +0500 Subject: [PATCH] Add broader exception. --- .../content/course_overviews/models.py | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/openedx/core/djangoapps/content/course_overviews/models.py b/openedx/core/djangoapps/content/course_overviews/models.py index 1c05bb4b98..0935ea3880 100644 --- a/openedx/core/djangoapps/content/course_overviews/models.py +++ b/openedx/core/djangoapps/content/course_overviews/models.py @@ -226,16 +226,12 @@ class CourseOverview(TimeStampedModel): ]) CourseOverviewImageSet.create_or_update(course_overview, course) - except IntegrityError: - # There is a rare race condition that will occur if - # CourseOverview.get_from_id is called while a - # another identical overview is already in the process - # of being created. - # One of the overviews will be saved normally, while the - # other one will cause an IntegrityError because it tries - # to save a duplicate. - # (see: https://openedx.atlassian.net/browse/TNL-2854). - pass + except Exception: # pylint: disable=broad-except + log.exception( + "CourseOverview for course %s failed!", + course_id, + ) + return course_overview elif course is not None: raise IOError(