Clean up

Author: Minh Nguyen

internal/venice-common/src/main/java/com/linkedin/venice/controllerapi/AdminOperationProtocolVersionControllerResponse.java

@@ -1,6 +1,5 @@
 package com.linkedin.venice.controllerapi;
 
-import com.fasterxml.jackson.annotation.JsonIgnore;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -33,25 +32,4 @@ public void setControllerNameToVersionMap(Map<String, Long> controllerNameToVers
   public Map<String, Long> getControllerNameToVersionMap() {
     return controllerNameToVersionMap;
   }
-
-  /*
-   * This method is used to convert the object to a string representation.
-   * AdminOperationProtocolVersionControllerResponse {"localAdminOperationProtocolVersion": 4, "cluster": "venice0,
-   * "requestUrl": "host7", "controllerUrlToVersionMap": {host7=4, host9=6, host8=5}}
-   */
-  @JsonIgnore
-  public String toString() {
-    return new StringBuilder().append(this.getClass().getSimpleName())
-        .append(" {")
-        .append("\"localAdminOperationProtocolVersion\": ")
-        .append(localAdminOperationProtocolVersion)
-        .append(", \"cluster\": \"")
-        .append(getCluster())
-        .append(", \"localControllerName\": \"")
-        .append(localControllerName)
-        .append("\", \"controllerNameToVersionMap\": ")
-        .append(controllerNameToVersionMap.toString())
-        .append("}")
-        .toString();
-  }
 }

internal/venice-test-common/src/integrationTest/java/com/linkedin/venice/controller/TestAdminOperationVersionDetection.java

@@ -1,6 +1,6 @@
 package com.linkedin.venice.controller;
 
-import static com.linkedin.venice.ConfigKeys.*;
+import static com.linkedin.venice.ConfigKeys.CONTROLLER_SSL_ENABLED;
 import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertFalse;