diff --git a/java/google/registry/model/registry/label/PremiumListUtils.java b/java/google/registry/model/registry/label/PremiumListUtils.java index bc05508cd..46506fb86 100644 --- a/java/google/registry/model/registry/label/PremiumListUtils.java +++ b/java/google/registry/model/registry/label/PremiumListUtils.java @@ -180,10 +180,8 @@ public final class PremiumListUtils { ofy().save().entities(newList, newRevision); return newList; }); - // Update the cache. - cachePremiumLists.put(premiumList.getName(), updated); - // Delete the entities under the old PremiumList. - oldPremiumList.ifPresent(PremiumListUtils::deleteRevisionAndEntriesOfPremiumList); + // TODO(b/79888775): Enqueue the oldPremiumList for deletion after at least + // RegistryConfig.getDomainLabelListCacheDuration() has elapsed. return updated; } diff --git a/javatests/google/registry/model/registry/label/PremiumListUtilsTest.java b/javatests/google/registry/model/registry/label/PremiumListUtilsTest.java index 214a37a9f..d49deb1d4 100644 --- a/javatests/google/registry/model/registry/label/PremiumListUtilsTest.java +++ b/javatests/google/registry/model/registry/label/PremiumListUtilsTest.java @@ -244,20 +244,7 @@ public class PremiumListUtilsTest { assertThat(getPremiumPrice("genius", registry)).hasValue(Money.parse("USD 10")); assertThat(getPremiumPrice("savant", registry)).hasValue(Money.parse("USD 90")); assertThat(getPremiumPrice("dolt", registry)).isEmpty(); - assertThat(ofy() - .load() - .type(PremiumListEntry.class) - .parent(pl.getRevisionKey()) - .id("dolt") - .now()) - .isNull(); - assertThat(ofy() - .load() - .type(PremiumListEntry.class) - .parent(pl2.getRevisionKey()) - .id("dolt") - .now()) - .isNull(); + // TODO(b/79888775): Assert that the old premium list is enqueued for later deletion. assertThat(premiumListChecks) .hasValueForLabels(4, "tld", "tld", UNCACHED_POSITIVE.toString()) .and()