Automated rollback of commit ba5bf5f.

*** Reason for rollback ***

Uses ImmutableSet that breaks validation of targets when it attempts to remove the target from the immutable iterator when invalid.

*** Original change description ***

Add scope validation for GraphlessBlazeQueryEnvironment.

PiperOrigin-RevId: 346365154

src/main/java/com/google/devtools/build/lib/query2/common/AbstractBlazeQueryEnvironment.java

@@ -16,7 +16,6 @@
 import static java.util.concurrent.TimeUnit.MINUTES;
 
 import com.google.common.base.Preconditions;
-import com.google.common.base.Predicates;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Iterables;
@@ -46,7 +45,6 @@
 import com.google.devtools.build.lib.util.DetailedExitCode;
 import java.io.IOException;
 import java.util.Collection;
-import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
@@ -279,20 +277,6 @@ public Map<Label, Target> getTargets(Iterable<Label> labels)
     return resultBuilder.build();
   }
 
-  protected void validateScopeOfTargets(Set<Target> targets) throws QueryException {
-    // Sets.filter would be more convenient here, but can't deal with exceptions.
-    if (labelFilter != Predicates.<Label>alwaysTrue()) {
-      // The labelFilter is always true for bazel query; it's only used for genquery rules.
-      Iterator<Target> targetIterator = targets.iterator();
-      while (targetIterator.hasNext()) {
-        Target target = targetIterator.next();
-        if (!validateScope(target.getLabel(), strictScope)) {
-          targetIterator.remove();
-        }
-      }
-    }
-  }
-
   protected boolean validateScope(Label label, boolean strict) throws QueryException {
     if (!labelFilter.test(label)) {
       String error = String.format("target '%s' is not within the scope of the query", label);

src/main/java/com/google/devtools/build/lib/query2/query/BlazeQueryEnvironment.java

@@ -17,6 +17,7 @@
 
 import com.google.common.base.Preconditions;
 import com.google.common.base.Predicate;
+import com.google.common.base.Predicates;
 import com.google.common.collect.ImmutableList;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.cmdline.PackageIdentifier;
@@ -62,6 +63,7 @@
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.HashSet;
+import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
@@ -170,7 +172,17 @@ private void getTargetsMatchingPatternImpl(String pattern, Callback<Target> call
     // becomes quadratic in runtime.
     Set<Target> targets = new LinkedHashSet<>(resolvedTargetPatterns.get(pattern));
 
-    validateScopeOfTargets(targets);
+    // Sets.filter would be more convenient here, but can't deal with exceptions.
+    if (labelFilter != Predicates.<Label>alwaysTrue()) {
+      // The labelFilter is always true for bazel query; it's only used for genquery rules.
+      Iterator<Target> targetIterator = targets.iterator();
+      while (targetIterator.hasNext()) {
+        Target target = targetIterator.next();
+        if (!validateScope(target.getLabel(), strictScope)) {
+          targetIterator.remove();
+        }
+      }
+    }
 
     Set<PackageIdentifier> packages = CompactHashSet.create();
     for (Target target : targets) {

src/main/java/com/google/devtools/build/lib/query2/query/GraphlessBlazeQueryEnvironment.java

@@ -13,7 +13,6 @@
 // limitations under the License.
 package com.google.devtools.build.lib.query2.query;
 
-import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Sets;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.cmdline.PackageIdentifier;
@@ -154,8 +153,6 @@ public QueryTaskFuture<Void> getTargetsMatchingPattern(
 
   private void getTargetsMatchingPatternImpl(String pattern, Callback<Target> callback)
       throws QueryException, InterruptedException {
-    Set<Target> targets = ImmutableSet.copyOf(resolvedTargetPatterns.get(pattern));
-    validateScopeOfTargets(targets);
     callback.process(resolvedTargetPatterns.get(pattern));
   }