Clarify that ErrorInfoValue is a NotComparableSkyValue because we don't expect equality, not because there's no implementation.

PiperOrigin-RevId: 201191262

1 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java b/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
index a0c5108590..8254f2eaf3 100644
--- a/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
+++ b/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
@@ -179,10 +179,10 @@ public abstract class ValueWithMetadata implements SkyValue {
   /**
    * Implementation of {@link ValueWithMetadata} for the error case.
    *
-   * ErorInfo does not override equals(), so it may as well be marked NotComparableSkyValue.
+   * <p>Mark NotComparableSkyValue because it's unlikely that re-evaluation gives the same error.
    */
   private static final class ErrorInfoValue extends ValueWithMetadata
-          implements NotComparableSkyValue {
+      implements NotComparableSkyValue {
 
     private final ErrorInfo errorInfo;
     private final NestedSet<TaggedEvents> transitiveEvents;