Discourage passing NestedSets to SpawnAction.Builder#addInputs

This is equivalent to the deprecation of NestedSetBuilder#addAll(NestedSet).

--
MOS_MIGRATED_REVID=127336417

1 files changed, 9 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java b/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
index 8060eb0290..a458972bf3 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
@@ -681,6 +681,15 @@ public class SpawnAction extends AbstractAction implements ExecutionInfoSpecifie
       return this;
     }
 
+    /** @deprecated Use {@link #addTransitiveInputs} to avoid excessive memory use. */
+    @Deprecated
+    public Builder addInputs(NestedSet<Artifact> artifacts) {
+      // Do not delete this method, or else addInputs(Iterable) calls with a NestedSet argument
+      // will not be flagged.
+      inputsBuilder.addAll((Iterable<Artifact>) artifacts);
+      return this;
+    }
+
     /**
      * Adds transitive inputs to this action.
      */