From 7074bcff7d1fcba7230fec11ce1d155b6e3d51b1 Mon Sep 17 00:00:00 2001
From: larsrc <larsrc@google.com>
Date: Tue, 14 Dec 2021 10:20:11 -0800
Subject: [PATCH] Automated rollback of commit
 ed825a49ef18a87e75fcbc381bb9b656e83b04fb.

*** Reason for rollback ***

Main suspect for regressions in current Blaze canary.

*** Original change description ***

Add cumulative SpawnMetrics to the blaze.actions table

PiperOrigin-RevId: 416335592
---
 .../build/lib/actions/ActionResult.java       | 100 ------------------
 1 file changed, 100 deletions(-)

diff --git a/src/main/java/com/google/devtools/build/lib/actions/ActionResult.java b/src/main/java/com/google/devtools/build/lib/actions/ActionResult.java
index eebdbcea622149..9b700e26f4c01a 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ActionResult.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ActionResult.java
@@ -164,106 +164,6 @@ public Optional<Long> cumulativeCommandExecutionMemoryInKb() {
     return getCumulativeLong(SpawnResult::getMemoryInKb);
   }
 
-  /**
-   * Returns the cumulative spawns total time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsTotalTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().totalTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns parse time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsParseTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().parseTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns network time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsNetworkTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().networkTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns fetch time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsFetchTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().fetchTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns queue time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsQueueTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().queueTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns setup time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsSetupTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().setupTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns upload time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeSpawnsUploadTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().uploadTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns execution wall time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeExecutionWallTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().executionWallTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns process output time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeProcessOutputTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().processOutputsTime()));
-  }
-
-  /**
-   * Returns the cumulative spawns retry time for the {@link Action}.
-   *
-   * @return the cumulative measurement, or empty in case of execution errors or when the
-   *     measurement is not implemented for the current platform
-   */
-  public Optional<Duration> cumulativeRetryTime() {
-    return getCumulativeTime(s -> Optional.of(s.getMetrics().retryTime()));
-  }
-
   /**
    * Indicates whether all {@link Spawn}s executed locally or not.
    *