Add SparseDiagnostic index for _FindOrInsertDiagnosticsOutOfOrder

Seeing lots of NeedIndexError in the logs for /add_histograms/process with
Suite: ChromiumPerf/chromium_perf/sizes as well as others.
SparseDiagnostic._FindOrInsertDiagnosticsOutOfOrder() calls query(end_revision
>= rev - 1, test == test).order(-end_revision). This query requires an index
like (test, -end_revision).
This CL adds the required index and a debug log print.

Bug: chromium:906685
Change-Id: I90ed1b6ce5e1d8fc2a620f298cb2d24d0def1ca6
Reviewed-on: https://chromium-review.googlesource.com/c/1362081
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Ben Hayden <benjhayden@chromium.org>
2 files changed
tree: c5f8f279c9769fc63f97ccb53862d2fba65fdf24
  1. bin/
  2. catapult_build/
  3. common/
  4. dashboard/
  5. dependency_manager/
  6. devil/
  7. docs/
  8. experimental/
  9. firefighter/
  10. hooks/
  11. infra/
  12. netlog_viewer/
  13. systrace/
  14. telemetry/
  15. third_party/
  16. trace_processor/
  17. tracing/
  18. web_page_replay_go/
  19. .eslintignore
  20. .eslintrc
  21. .gitignore
  22. .vpython
  23. AUTHORS
  24. BUILD.gn
  25. codereview.settings
  26. CONTRIBUTING.md
  27. generate_telemetry_build.py
  28. LICENSE
  29. navbar.md
  30. OWNERS
  31. PRESUBMIT.py
  32. pylintrc
  33. README.md
  34. WATCHLISTS
README.md

Catapult

Catapult is the home for several performance tools that span from gathering, displaying and analyzing performance data. This includes:

These tools were created by Chromium developers for performance analysis, testing, and monitoring of Chrome, but they can also be used for analyzing and monitoring websites, and eventually Android apps.

Contributing

Please see our contributor's guide

Current build status