Remove dual-write of registrar from tool commands (#952)

* Remove dual-write of registrar from tool commands

As discussed, we're keeping registrar in the "replicated" category.

core/src/main/java/google/registry/tools/CreateOrUpdateRegistrarCommand.java

@@ -18,7 +18,6 @@ import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Predicates.isNull;
 import static com.google.common.base.Strings.isNullOrEmpty;
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
-import static google.registry.persistence.transaction.TransactionManagerFactory.jpaTm;
 import static google.registry.util.DomainNameUtils.canonicalizeDomainName;
 import static google.registry.util.RegistrarUtils.normalizeRegistrarName;
 import static java.nio.charset.StandardCharsets.US_ASCII;
@@ -29,7 +28,6 @@ import com.google.common.base.Joiner;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Sets;
-import com.google.common.flogger.FluentLogger;
 import google.registry.flows.certs.CertificateChecker;
 import google.registry.model.registrar.Registrar;
 import google.registry.model.registrar.RegistrarAddress;
@@ -57,8 +55,6 @@ import org.joda.time.DateTime;
 /** Shared base class for commands to create or update a {@link Registrar}. */
 abstract class CreateOrUpdateRegistrarCommand extends MutatingCommand {
 
-  static final FluentLogger logger = FluentLogger.forEnclosingClass();
-
   @Inject CertificateChecker certificateChecker;
 
   @Parameter(description = "Client identifier of the registrar account", required = true)
@@ -457,26 +453,4 @@ abstract class CreateOrUpdateRegistrarCommand extends MutatingCommand {
       stageEntityChange(oldRegistrar, newRegistrar);
     }
   }
-
-  @Override
-  protected String execute() throws Exception {
-    // Save registrar to Datastore and output its response
-    logger.atInfo().log(super.execute());
-
-    String cloudSqlMessage;
-    try {
-      jpaTm()
-          .transact(
-              () ->
-                  getChangedEntities().forEach(newEntity -> saveToCloudSql((Registrar) newEntity)));
-      cloudSqlMessage =
-          String.format("Updated %d entities in Cloud SQL.\n", getChangedEntities().size());
-    } catch (Throwable t) {
-      cloudSqlMessage = "Unexpected error saving registrar to Cloud SQL from nomulus tool command";
-      logger.atSevere().withCause(t).log(cloudSqlMessage);
-    }
-    return cloudSqlMessage;
-  }
-
-  abstract void saveToCloudSql(Registrar registrar);
 }

core/src/main/java/google/registry/tools/CreateRegistrarCommand.java

@@ -19,7 +19,6 @@ import static com.google.common.base.Preconditions.checkState;
 import static com.google.common.base.Strings.emptyToNull;
 import static com.google.common.collect.Iterables.getOnlyElement;
 import static google.registry.model.registrar.Registrar.State.ACTIVE;
-import static google.registry.persistence.transaction.TransactionManagerFactory.jpaTm;
 import static google.registry.tools.RegistryToolEnvironment.PRODUCTION;
 import static google.registry.tools.RegistryToolEnvironment.SANDBOX;
 import static google.registry.tools.RegistryToolEnvironment.UNITTEST;
@@ -70,11 +69,6 @@ final class CreateRegistrarCommand extends CreateOrUpdateRegistrarCommand
     registrarState = Optional.ofNullable(registrarState).orElse(ACTIVE);
   }
 
-  @Override
-  void saveToCloudSql(Registrar registrar) {
-    jpaTm().insert(registrar);
-  }
-
   @Nullable
   @Override
   Registrar getOldRegistrar(final String clientId) {

core/src/main/java/google/registry/tools/UpdateRegistrarCommand.java

@@ -14,7 +14,6 @@
 
 package google.registry.tools;
 
-import static google.registry.persistence.transaction.TransactionManagerFactory.jpaTm;
 import static google.registry.util.PreconditionsUtils.checkArgumentNotNull;
 import static google.registry.util.PreconditionsUtils.checkArgumentPresent;
 
@@ -50,9 +49,4 @@ final class UpdateRegistrarCommand extends CreateOrUpdateRegistrarCommand {
               + " contact.");
     }
   }
-
-  @Override
-  void saveToCloudSql(Registrar registrar) {
-    jpaTm().update(registrar);
-  }
 }