[Predator] Abort publishing result when the analysis failed.

appengine/findit/crash/crash_pipeline.py

Index: appengine/findit/crash/crash_pipeline.py
diff --git a/appengine/findit/crash/crash_pipeline.py b/appengine/findit/crash/crash_pipeline.py
index 52ef7a4d10d9942ba325519923862938ac69ac76..753e9e7ba2552af6b5bb8c0dcb8016972b8d0e35 100644
--- a/appengine/findit/crash/crash_pipeline.py
+++ b/appengine/findit/crash/crash_pipeline.py
@@ -179,6 +179,11 @@ class PublishResultPipeline(CrashBasePipeline):
     (except for ``self``, naturally).
     """
     analysis = self._findit.GetAnalysis(self._crash_identifiers)
+    if analysis.failed:
+      logging.info('Analysis of %s failed, abort publishing result to %s',
+                   repr(self._crash_identifiers), self.client_id)

[wrengr, 2016/11/22 18:37:49]

Since the crash identifiers might be long, I'd rephrase things to put it at the
end. Maybe something like "Can't publish result to %s because analysis failed:
%s" ?

[Sharu Jiang, 2016/11/22 18:58:56]

Done.

+      return
+
     result = analysis.ToPublishableResult(self._crash_identifiers)
     messages_data = [json.dumps(result, sort_keys=True)]