Skip to content

Commit

Permalink
Rename the compact execution log to better reflect the file format.
Browse files Browse the repository at this point in the history
Closes #22172.

PiperOrigin-RevId: 629494462
Change-Id: Ib1febd4244c2a919969c125a96512d13cf6d6a6b
  • Loading branch information
sluongng authored and Copybara-Service committed Apr 30, 2024
1 parent ff40449 commit 0e7c0fb
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@

/** Module providing on-demand spawn logging. */
public final class SpawnLogModule extends BlazeModule {
private static final String EXEC_LOG_FILENAME = "execution_log.binpb.zstd";

@Nullable private SpawnLogContext spawnLogContext;
@Nullable private Path outputPath;
Expand Down Expand Up @@ -123,7 +124,7 @@ private void initOutputs(CommandEnvironment env) throws IOException {

// Use a well-known temporary path to avoid accumulation of potentially large files in /tmp
// due to abnormally terminated invocations (e.g., when running out of memory).
Path tempPath = outputBase.getRelative("execution.log");
Path tempPath = outputBase.getRelative(EXEC_LOG_FILENAME);

spawnLogContext =
new ExpandedSpawnLogContext(
Expand Down Expand Up @@ -182,7 +183,7 @@ public void buildComplete(BuildCompleteEvent event) {
try {
spawnLogContext.close();
if (spawnLogContext.shouldPublish()) {
event.getResult().getBuildToolLogCollection().addLocalFile("execution.log", outputPath);
event.getResult().getBuildToolLogCollection().addLocalFile(EXEC_LOG_FILENAME, outputPath);
}
} catch (IOException e) {
abruptExit =
Expand Down

0 comments on commit 0e7c0fb

Please sign in to comment.