From ab9b7c613d5a3fca6d655e1d2d0089d68e27f506 Mon Sep 17 00:00:00 2001 From: nickfelt Date: Thu, 23 Mar 2017 10:22:01 -0700 Subject: [PATCH] Remove now-unused RegistryData SQL columns These columns were only necessary to support the old PremiumListData and RecurringEventData views, which were removed in [] ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=151019824 --- .../registry/tools/sql/registry_data_view.sql | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/java/google/registry/tools/sql/registry_data_view.sql b/java/google/registry/tools/sql/registry_data_view.sql index 0087c5b94..861049231 100644 --- a/java/google/registry/tools/sql/registry_data_view.sql +++ b/java/google/registry/tools/sql/registry_data_view.sql @@ -21,22 +21,6 @@ SELECT tldStr AS tld, tldType AS type, - -- This relies on the fact that we currently haven't ever used more than one - -- value with renewBillingCostTransitions. - -- TODO(b/20764952): fix this limitation. - FIRST(renewBillingCostTransitions.billingCost) WITHIN RECORD - AS renewBillingCost, - -- Grace period lengths are stored in the ISO 8601 duration format as a - -- duration in seconds, which produces a string of the form "PT####S". - INTEGER(REGEXP_EXTRACT(autoRenewGracePeriodLength, r'PT(.+)S')) - AS autorenewGracePeriodSeconds, - premiumList.path AS premiumListPath, - -- TODO(b/18265521): remove this column once the referenced bug is fixed; - -- this column is only included because without it BigQuery will complain - -- that we aren't consuming reservedLists (due to it having a repeated .path - -- child field, which overlaps with premiumList.path above). - GROUP_CONCAT_UNQUOTED(reservedLists.path, "/") WITHIN RECORD - AS reservedListPaths, FROM [%SOURCE_DATASET%.Registry] ORDER BY