The benefit of "raise" on its own is that the original stacktrace is …

…made available, rather than creating a new stacktrace with "raise search_error". So its easier to debug.
ckan · Mar 30, 2016 · 617f6e2 · 617f6e2
1 parent 3bd3e2d
commit 617f6e2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ckan/model/modification.py b/ckan/model/modification.py
@@ -88,7 +88,7 @@ def notify(self, entity, operation):
                 log.exception(search_error)
                 # Reraise, since it's pretty crucial to ckan if it can't index
                 # a dataset
-                raise search_error
+                raise
             except Exception, ex:
                 log.exception(ex)
                 # Don't reraise other exceptions since they are generally of
@@ -103,7 +103,7 @@ def notify_after_commit(self, entity, operation):
                 log.exception(search_error)
                 # Reraise, since it's pretty crucial to ckan if it can't index
                 # a dataset
-                raise search_error
+                raise
             except Exception, ex:
                 log.exception(ex)
                 # Don't reraise other exceptions since they are generally of