Skip to content

Commit

Permalink
For logging purposes, consider the action local if at least one of it…
Browse files Browse the repository at this point in the history
…s spawns was local.

Otherwise, the logs are useless to determine the efficacy of dynamically executed actions, which correspond to a (local, remote) spawn pair.

PiperOrigin-RevId: 656379340
Change-Id: I2779a72fa5ca868dfc90e91d5b5baf69a4e8ddfe
  • Loading branch information
tjgq authored and copybara-github committed Jul 26, 2024
1 parent 1a05eda commit dbc59fe
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -250,14 +250,14 @@ public int cumulativeRetryTimeInMs() {
}

/**
* Indicates whether all {@link Spawn}s executed locally or not.
* Indicates whether the action had at least one locally executed spawn.
*
* @return true if all spawns of action executed locally
* @return true if at the action had at least one locally executed spawn
*/
public boolean locallyExecuted() {
boolean locallyExecuted = true;
boolean locallyExecuted = false;
for (SpawnResult spawnResult : spawnResults()) {
locallyExecuted &= !spawnResult.wasRemote();
locallyExecuted |= !spawnResult.wasRemote();
}
return locallyExecuted;
}
Expand Down

0 comments on commit dbc59fe

Please sign in to comment.