From b525bb6ebe8f45179b641a6f3efdbd8a81c6f7eb Mon Sep 17 00:00:00 2001 From: Tianyu Geng Date: Fri, 2 Jun 2023 18:05:43 -0700 Subject: [PATCH] Minor performance improvement with `--useCquery` Computation transitions is slow and it's not necessary when trying to figure out list of compatible targets. --- .../main/kotlin/com/bazel_diff/bazel/BazelQueryService.kt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cli/src/main/kotlin/com/bazel_diff/bazel/BazelQueryService.kt b/cli/src/main/kotlin/com/bazel_diff/bazel/BazelQueryService.kt index a43d519..e10feb0 100644 --- a/cli/src/main/kotlin/com/bazel_diff/bazel/BazelQueryService.kt +++ b/cli/src/main/kotlin/com/bazel_diff/bazel/BazelQueryService.kt @@ -75,7 +75,10 @@ class BazelQueryService( addAll(startupOptions) if (useCquery) { add("cquery") - add("--transitions=lite") + if (!outputCompatibleTargets) { + // There is no need to query the transitions when querying for compatible targets. + add("--transitions=lite") + } } else { add("query") }