From 84dd8c129ba4b24a58f0159adc9c93286a02176f Mon Sep 17 00:00:00 2001 From: Googler Date: Mon, 22 Jan 2024 12:11:19 -0800 Subject: [PATCH] Update `ClientEnvironmentFunction.Key` to use `@AutoCodec.Interner` instead of `@AutoCodec.Instantiator`. PiperOrigin-RevId: 600530886 Change-Id: I7f4bed967ef993309ea83585fc05d9d05fd221ab --- .../build/lib/skyframe/ClientEnvironmentFunction.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ClientEnvironmentFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/ClientEnvironmentFunction.java index b996d8292220d4..17871773d4e0b4 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/ClientEnvironmentFunction.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/ClientEnvironmentFunction.java @@ -41,12 +41,16 @@ private Key(String arg) { super(arg); } - @VisibleForSerialization - @AutoCodec.Instantiator - static Key create(String arg) { + private static Key create(String arg) { return interner.intern(new Key(arg)); } + @VisibleForSerialization + @AutoCodec.Interner + static Key intern(Key key) { + return interner.intern(key); + } + @Override public SkyFunctionName functionName() { return SkyFunctions.CLIENT_ENVIRONMENT_VARIABLE;