From 577cfbc8a2b0841cd67a918febe9e3c5fcb3882e Mon Sep 17 00:00:00 2001
From: Benjamin Peterson <benjamin@engflow.com>
Date: Fri, 20 May 2022 07:24:14 -0700
Subject: [PATCH] Set --legacy_important_outputs to false by default.

This flag reduces the largest proto size, which helps avoid sharp edges with remote execution systems (e.g., https://github.com/bazelbuild/bazel/issues/12050).

RELNOTES[INC]: --legacy_important_outputs now has a default of false.

Closes #14353.

PiperOrigin-RevId: 449979885
---
 .../BuildEventProtocolOptions.java                 | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventProtocolOptions.java b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventProtocolOptions.java
index 2ffb42fade668b..e925e46d4efc87 100644
--- a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventProtocolOptions.java
+++ b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventProtocolOptions.java
@@ -23,13 +23,13 @@
 public class BuildEventProtocolOptions extends OptionsBase {
 
   @Option(
-    name = "legacy_important_outputs",
-    defaultValue = "true",
-    documentationCategory = OptionDocumentationCategory.LOGGING,
-    effectTags = {OptionEffectTag.AFFECTS_OUTPUTS},
-    help = "Use this to suppress generation of the legacy important_outputs field in the "
-        + "TargetComplete event."
-  )
+      name = "legacy_important_outputs",
+      defaultValue = "false",
+      documentationCategory = OptionDocumentationCategory.LOGGING,
+      effectTags = {OptionEffectTag.AFFECTS_OUTPUTS},
+      help =
+          "Use this to suppress generation of the legacy important_outputs field in the "
+              + "TargetComplete event.")
   public boolean legacyImportantOutputs;
 
   @Option(