Refactor TargetPatternPhaseFunction

- Make TargetPatternPhaseKey implement SkyKey
- Move the TargetParsingCompleteEvent posting into the function
- Split the time reporting out into TargetParsingPhaseTimeEvent

PiperOrigin-RevId: 162475743

1 files changed, 0 insertions, 8 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/pkgcache/TargetParsingCompleteEvent.java b/src/main/java/com/google/devtools/build/lib/pkgcache/TargetParsingCompleteEvent.java
index 2a0763b8fc..64d4eb1f1e 100644
--- a/src/main/java/com/google/devtools/build/lib/pkgcache/TargetParsingCompleteEvent.java
+++ b/src/main/java/com/google/devtools/build/lib/pkgcache/TargetParsingCompleteEvent.java
@@ -36,7 +36,6 @@ public class TargetParsingCompleteEvent implements BuildEvent {
   private final ImmutableSet<Target> filteredTargets;
   private final ImmutableSet<Target> testFilteredTargets;
   private final ImmutableSet<Target> expandedTargets;
-  private final long timeInMs;
 
   /**
    * Construct the event.
@@ -47,10 +46,8 @@ public class TargetParsingCompleteEvent implements BuildEvent {
       Collection<Target> targets,
       Collection<Target> filteredTargets,
       Collection<Target> testFilteredTargets,
-      long timeInMs,
       List<String> originalTargetPattern,
       Collection<Target> expandedTargets) {
-    this.timeInMs = timeInMs;
     this.targets = ImmutableSet.copyOf(targets);
     this.filteredTargets = ImmutableSet.copyOf(filteredTargets);
     this.testFilteredTargets = ImmutableSet.copyOf(testFilteredTargets);
@@ -64,7 +61,6 @@ public class TargetParsingCompleteEvent implements BuildEvent {
         targets,
         ImmutableSet.<Target>of(),
         ImmutableSet.<Target>of(),
-        0,
         ImmutableList.<String>of(),
         targets);
   }
@@ -94,10 +90,6 @@ public class TargetParsingCompleteEvent implements BuildEvent {
     return testFilteredTargets;
   }
 
-  public long getTimeInMs() {
-    return timeInMs;
-  }
-
   @Override
   public BuildEventId getEventId() {
     return BuildEventId.targetPatternExpanded(originalTargetPattern);