diff options
author | 2017-02-24 16:30:15 +0000 | |
---|---|---|
committer | 2017-02-27 15:05:00 +0000 | |
commit | 777b30d06700f76ba580715429f3663de3fa0529 (patch) | |
tree | d3abdc6f42d04256c486270add24e5b0c5d3abbd /src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java | |
parent | 25aa033ad5657a5cfa16e8307464648b9374be2d (diff) |
Provide more reporting options to SkyFunctions
With more specific information to be reported by Skyfunctions, e.g.,
to inform the build-event protocol on missing files, the EventHandler
interface is no longer enough. Therefore, provide an enriched context
for reporting events.
--
Change-Id: I2d06166fe4d5b9054e24ad8c752fafc039e3f9f8
Reviewed-on: https://cr.bazel.build/8794
PiperOrigin-RevId: 148463437
MOS_MIGRATED_REVID=148463437
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java b/src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java index eb39b01f95..867ec3d8bf 100644 --- a/src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java +++ b/src/main/java/com/google/devtools/build/lib/query2/BlazeQueryEnvironment.java @@ -22,7 +22,7 @@ import com.google.devtools.build.lib.cmdline.Label; import com.google.devtools.build.lib.cmdline.LabelSyntaxException; import com.google.devtools.build.lib.cmdline.ResolvedTargets; import com.google.devtools.build.lib.cmdline.TargetParsingException; -import com.google.devtools.build.lib.events.EventHandler; +import com.google.devtools.build.lib.events.ExtendedEventHandler; import com.google.devtools.build.lib.graph.Digraph; import com.google.devtools.build.lib.graph.Node; import com.google.devtools.build.lib.packages.Attribute; @@ -89,22 +89,24 @@ public class BlazeQueryEnvironment extends AbstractBlazeQueryEnvironment<Target> /** * Note that the correct operation of this class critically depends on the Reporter being a * singleton object, shared by all cooperating classes contributing to Query. + * * @param strictScope if true, fail the whole query if a label goes out of scope. - * @param loadingPhaseThreads the number of threads to use during loading - * the packages for the query. - * @param labelFilter a predicate that determines if a specific label is - * allowed to be visited during query execution. If it returns false, - * the query execution is stopped with an error message. + * @param loadingPhaseThreads the number of threads to use during loading the packages for the + * query. + * @param labelFilter a predicate that determines if a specific label is allowed to be visited + * during query execution. If it returns false, the query execution is stopped with an error + * message. * @param settings a set of enabled settings */ - BlazeQueryEnvironment(TransitivePackageLoader transitivePackageLoader, + BlazeQueryEnvironment( + TransitivePackageLoader transitivePackageLoader, TargetProvider targetProvider, TargetPatternEvaluator targetPatternEvaluator, boolean keepGoing, boolean strictScope, int loadingPhaseThreads, Predicate<Label> labelFilter, - EventHandler eventHandler, + ExtendedEventHandler eventHandler, Set<Setting> settings, Iterable<QueryFunction> extraFunctions, QueryExpressionEvalListener<Target> evalListener) { |