crash: Avoid redundant analysis of crashes from Fracas |
|||||
Issue descriptionFracas keeps producing crash data for Findit to analyze. However, not all crashes of the same (channel, platform, signature) has to be re-analyzed each time. If the stack trace and regression range do not change, the analysis result will be the same. We'd better avoid such redundant analysis, and save resource to analyze more crashes in parallel. Another factor to consider is the Findit version: if we upgrade our algorithm, the result might change even the crash data is the same.
,
May 26 2016
Currently, if a crash with the same crash identifiers is already analyzed, the crash won't be analyzed anymore. However, we need to make Findit smarter. For example, if regression range changes, the crash should be re-analyzed. Or have an expired time for the analysis results.
,
Sep 28 2016
,
Oct 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/c958b3053825c8e2e4f3b9a99b31446e826ae22d commit c958b3053825c8e2e4f3b9a99b31446e826ae22d Author: katesonia <katesonia@chromium.org> Date: Fri Oct 07 22:37:00 2016 [Findit] Rerun if the regression range is different. Since stacktraces are already grouped at crash side, findit needs not to duplicate this check, so only add regression range check for smart rerun. BUG= 600535 Review-Url: https://codereview.chromium.org/2378133004 [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/crash_pipeline.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/detect_regression_range.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/findit_for_chromecrash.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/findit_for_client.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/test/crash_pipeline_test.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/test/findit_for_chromecrash_test.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/crash/test/findit_for_client_test.py [modify] https://crrev.com/c958b3053825c8e2e4f3b9a99b31446e826ae22d/appengine/findit/model/crash/crash_analysis.py
,
Oct 7 2016
,
Nov 30 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by st...@chromium.org
, Apr 21 2016Labels: -Findit
Summary: crash: Avoid redundant analysis of crashes from Fracas (was: Avoid redundant analysis of crashes from Fracas)