Automated rollback of commit b9544d0bdb2e3515328bb571d5c2ad871f0fe9c3.
*** Reason for rollback ***
Performance regression in nightly on yt-web b/211971772
*** Original change description ***
Use literal as a progress message in proto_common call.
This should save some memory allocations.
PiperOrigin-RevId: 419593344
diff --git a/src/main/java/com/google/devtools/build/lib/rules/proto/ProtoCompileActionBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/proto/ProtoCompileActionBuilder.java
index f6083b1..227917c 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/proto/ProtoCompileActionBuilder.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/proto/ProtoCompileActionBuilder.java
@@ -65,7 +65,7 @@
private final ProtoInfo protoInfo;
private final FilesToRunProvider protoCompiler;
- private final String progressMessage;
+ private final String language;
private final String langPrefix;
private final Iterable<Artifact> outputs;
private Iterable<Artifact> inputs;
@@ -122,12 +122,12 @@
public ProtoCompileActionBuilder(
ProtoInfo protoInfo,
FilesToRunProvider protoCompiler,
- String progressMessage,
+ String language,
String langPrefix,
Iterable<Artifact> outputs) {
this.protoInfo = protoInfo;
this.protoCompiler = protoCompiler;
- this.progressMessage = progressMessage;
+ this.language = language;
this.langPrefix = langPrefix;
this.outputs = outputs;
this.mnemonic = DEFAULT_MNEMONIC;
@@ -207,7 +207,7 @@
/* ctx */ ruleContext.getStarlarkRuleContext(),
/* proto_info */ protoInfo,
/* proto_compiler */ protoCompiler,
- /* progress_message */ progressMessage,
+ /* progress_message */ String.format("Generating %s proto_library %%{label}", language),
/* outputs */ StarlarkList.immutableCopyOf(outputs),
/* additional_args */ StarlarkList.immutableCopyOf(additionalArgs.build()),
/* plugins */ StarlarkList.immutableCopyOf(plugins.build()),
