From f745e99db7632cfb2145b6926f961e85f9084bc5 Mon Sep 17 00:00:00 2001
From: Janak Ramakrishnan <janakr@google.com>
Date: Thu, 3 Mar 2016 08:08:50 +0000
Subject: Use static creation method for SkyKey. This allows interning SkyKeys
 as they are created, as opposed to when they are requested from the
 ParallelEvaluator. That delay can lead to large memory spikes and churn.

--
MOS_MIGRATED_REVID=116224565
---
 .../com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/main/java/com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java')

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java b/src/main/java/com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java
index 82f4c559e8..dfff25096b 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/ChainUniquenessUtils.java
@@ -33,7 +33,7 @@ class ChainUniquenessUtils {
    */
   static SkyKey key(SkyFunctionName functionName, ImmutableList<? extends Object> chain) {
     Preconditions.checkState(!chain.isEmpty());
-    return new SkyKey(functionName, canonicalize(chain));
+    return SkyKey.create(functionName, canonicalize(chain));
   }
 
   private static ImmutableList<Object> canonicalize(ImmutableList<? extends Object> cycle) {
-- 
cgit v1.2.3