Patchwork D12397: search-discovery-case: display more information about the interresting case

login
register
mail settings
Submitter phabricator
Date March 22, 2022, 7:06 a.m.
Message ID <differential-rev-PHID-DREV-gfbdlna6c33yqef75nci-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/50741/
State New
Headers show

Comments

phabricator - March 22, 2022, 7:06 a.m.
marmoute created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  We display information about the total number of revs and the common/missing
  numbers. This is useful to spot the interresting case.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D12397

AFFECTED FILES
  contrib/perf-utils/search-discovery-case

CHANGE DETAILS




To: marmoute, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/contrib/perf-utils/search-discovery-case b/contrib/perf-utils/search-discovery-case
--- a/contrib/perf-utils/search-discovery-case
+++ b/contrib/perf-utils/search-discovery-case
@@ -147,13 +147,23 @@ 
     roundtrips = res["total-roundtrips"]
     if roundtrips <= 1:
         return None
+    total_rev = res["nb-revs"]
+    common_rev = res["nb-revs-common"]
+    missing_rev = res["nb-revs-missing"]
     undecided_common = res["nb-ini_und-common"]
     undecided_missing = res["nb-ini_und-missing"]
     if undecided_common == 0:
         return None
     if undecided_missing == 0:
         return None
-    return (roundtrips, undecided_common, undecided_missing)
+    return (
+        roundtrips,
+        undecided_common,
+        undecided_missing,
+        total_rev,
+        common_rev,
+        missing_rev,
+    )
 
 
 def end(*args, **kwargs):