| commit | 64976c6c20a10043d98b094fa256695d789c10b3 | [log] [tgz] |
|---|---|---|
| author | Googler <noreply@google.com> | Fri Aug 24 14:51:36 2018 -0700 |
| committer | Copybara-Service <copybara-piper@google.com> | Fri Aug 24 14:53:21 2018 -0700 |
| tree | ee8c46f3936807c803c7f38eec5b3870ec325707 | |
| parent | c3283a3fedd4d520457fb15d4974ce4fe979807e [diff] |
Hoist call to ImmutableMap.copyOf(). RELNOTES: None. PiperOrigin-RevId: 210161137
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeActionExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeActionExecutor.java index 56154e4..e466704 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeActionExecutor.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeActionExecutor.java
@@ -361,7 +361,8 @@ } public void setClientEnv(Map<String, String> clientEnv) { - this.clientEnv = clientEnv; + // Copy once here, instead of on every construction of ActionExecutionContext. + this.clientEnv = ImmutableMap.copyOf(clientEnv); } boolean usesActionFileSystem() {