Make BuildDriverKey non-CPU heavy.

Previously, a heavy-lifting part of conflict checking was done in the skyframe pool (collection of the action lookup values), it made sense to limit the concurrency by running it in the CPU-heavy pool. Now that we perform the entire conflict checking in an extra-Skyframe location, BuildDriverKey should no longer be CPU-heavy.

PiperOrigin-RevId: 561649161
Change-Id: Ic76661389d856ef9c492bd7b921649fa4455b583

src/main/java/com/google/devtools/build/lib/skyframe/BUILD

@@ -931,7 +931,6 @@ java_library(
         ":sky_functions",
         "//src/main/java/com/google/devtools/build/lib/actions:action_lookup_key",
         "//src/main/java/com/google/devtools/build/lib/analysis:top_level_artifact_context",
-        "//src/main/java/com/google/devtools/build/skyframe:cpu_heavy_skykey",
         "//src/main/java/com/google/devtools/build/skyframe:skyframe-objects",
     ],
 )

src/main/java/com/google/devtools/build/lib/skyframe/BuildDriverKey.java

@@ -15,15 +15,15 @@
 
 import com.google.devtools.build.lib.actions.ActionLookupKey;
 import com.google.devtools.build.lib.analysis.TopLevelArtifactContext;
-import com.google.devtools.build.skyframe.CPUHeavySkyKey;
 import com.google.devtools.build.skyframe.SkyFunctionName;
+import com.google.devtools.build.skyframe.SkyKey;
 import java.util.Objects;
 
 /**
  * Wraps an {@link ActionLookupKey}. The evaluation of this SkyKey is the entry point of analyzing
  * the {@link ActionLookupKey} and executing the associated actions.
  */
-public final class BuildDriverKey implements CPUHeavySkyKey {
+public final class BuildDriverKey implements SkyKey {
   private final ActionLookupKey actionLookupKey;
   private final TopLevelArtifactContext topLevelArtifactContext;
   private final boolean strictActionConflictCheck;