Use Immutable types for static final fields

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=149260759
This commit is contained in:
Ben McIlwain 2017-03-05 20:04:40 -08:00
parent 439742fd11
commit 0cbc2bdab6
3 changed files with 12 additions and 12 deletions

View file

@ -79,7 +79,8 @@ public final class TldFanoutAction implements Runnable {
private static final String JITTER_SECONDS_PARAM = "jitterSeconds"; private static final String JITTER_SECONDS_PARAM = "jitterSeconds";
/** A set of control params to TldFanoutAction that aren't passed down to the executing action. */ /** A set of control params to TldFanoutAction that aren't passed down to the executing action. */
private static final Set<String> CONTROL_PARAMS = ImmutableSet.of( private static final ImmutableSet<String> CONTROL_PARAMS =
ImmutableSet.of(
ENDPOINT_PARAM, ENDPOINT_PARAM,
QUEUE_PARAM, QUEUE_PARAM,
FOR_EACH_REAL_TLD_PARAM, FOR_EACH_REAL_TLD_PARAM,

View file

@ -19,11 +19,11 @@ import static com.google.common.base.Predicates.not;
import static java.util.Arrays.asList; import static java.util.Arrays.asList;
import com.google.common.base.Function; import com.google.common.base.Function;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.Iterables; import com.google.common.collect.Iterables;
import com.google.common.collect.Maps; import com.google.common.collect.Maps;
import google.registry.model.translators.EnumToAttributeAdapter; import google.registry.model.translators.EnumToAttributeAdapter;
import google.registry.model.translators.EnumToAttributeAdapter.EppEnum; import google.registry.model.translators.EnumToAttributeAdapter.EppEnum;
import java.util.Map;
import javax.annotation.Nullable; import javax.annotation.Nullable;
import javax.xml.bind.annotation.XmlAttribute; import javax.xml.bind.annotation.XmlAttribute;
import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter; import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
@ -100,7 +100,7 @@ public enum GracePeriodStatus implements EppEnum {
TRANSFER("transferPeriod"); TRANSFER("transferPeriod");
/** Provide a quick lookup of GracePeriodStatus from XML name. */ /** Provide a quick lookup of GracePeriodStatus from XML name. */
private static final Map<String, GracePeriodStatus> XML_NAME_TO_GRACE_PERIOD_STATUS = private static final ImmutableMap<String, GracePeriodStatus> XML_NAME_TO_GRACE_PERIOD_STATUS =
Maps.uniqueIndex( Maps.uniqueIndex(
// SUNRUSH_ADD isn't a real grace period type visible in EPP XML, so exclude it. // SUNRUSH_ADD isn't a real grace period type visible in EPP XML, so exclude it.
Iterables.filter(asList(GracePeriodStatus.values()), not(equalTo(SUNRUSH_ADD))), Iterables.filter(asList(GracePeriodStatus.values()), not(equalTo(SUNRUSH_ADD))),

View file

@ -39,7 +39,6 @@ import google.registry.request.ParameterMap;
import google.registry.util.FormattingLogger; import google.registry.util.FormattingLogger;
import java.io.IOException; import java.io.IOException;
import java.util.Map; import java.util.Map;
import java.util.Set;
import javax.inject.Inject; import javax.inject.Inject;
/** Action for exporting metrics to BigQuery. */ /** Action for exporting metrics to BigQuery. */
@ -49,7 +48,7 @@ public class MetricsExportAction implements Runnable {
public static final String PATH = "/_dr/task/metrics"; public static final String PATH = "/_dr/task/metrics";
private static final FormattingLogger logger = getLoggerForCallerClass(); private static final FormattingLogger logger = getLoggerForCallerClass();
private static final String DATASET_ID = "metrics"; private static final String DATASET_ID = "metrics";
private static final Set<String> SPECIAL_PARAMS = ImmutableSet.of("tableId", "insertId"); private static final ImmutableSet<String> SPECIAL_PARAMS = ImmutableSet.of("tableId", "insertId");
@Inject @Parameter("tableId") String tableId; @Inject @Parameter("tableId") String tableId;
@Inject @Parameter("insertId") String insertId; @Inject @Parameter("insertId") String insertId;