Removed unreferenced code (#5935)

Author: dlmarion

core/src/main/java/org/apache/accumulo/core/cli/ClientOpts.java

@@ -170,9 +170,6 @@ public AuthenticationToken getToken() {
       + "accumulo-client.properties. Expected format: -o <key>=<value>")
   private List<String> overrides = new ArrayList<>();
 
-  @Parameter(names = "--trace", description = "turn on distributed tracing")
-  public boolean trace = false;
-
   public Map<String,String> getOverrides() {
     return ConfigOpts.getOverrides(overrides);
   }

core/src/main/java/org/apache/accumulo/core/client/NamespaceNotEmptyException.java

@@ -18,8 +18,6 @@
  */
 package org.apache.accumulo.core.client;
 
-import org.apache.accumulo.core.clientImpl.thrift.ThriftTableOperationException;
-
 /**
  * Thrown when the namespace specified contains tables
  */
@@ -55,13 +53,6 @@ public NamespaceNotEmptyException(String namespaceId, String namespaceName, Stri
     super.initCause(cause);
   }
 
-  /**
-   * @param e constructs an exception from a thrift exception
-   */
-  public NamespaceNotEmptyException(ThriftTableOperationException e) {
-    this(e.getTableId(), e.getTableName(), e.getDescription(), e);
-  }
-
   /**
    * @return the name of the namespace
    */

core/src/main/java/org/apache/accumulo/core/client/admin/NewTableConfiguration.java

@@ -272,7 +272,6 @@ public NewTableConfiguration setLocalityGroups(Map<String,Set<Text>> groups) {
    *
    * @since 2.0.0
    */
-  @SuppressWarnings("unchecked")
   public NewTableConfiguration withSplits(final SortedSet<Text> splits) {
     checkArgument(splits != null, "splits set is null");
     checkArgument(!splits.isEmpty(), "splits set is empty");

core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java

@@ -1143,10 +1143,6 @@ public void setProperty(ClientProperty property, CharSequence value) {
       properties.setProperty(property.getKey(), value.toString());
     }
 
-    public void setProperty(ClientProperty property, Long value) {
-      setProperty(property, Long.toString(value));
-    }
-
     public void setProperty(ClientProperty property, Integer value) {
       setProperty(property, Integer.toString(value));
     }

core/src/main/java/org/apache/accumulo/core/clientImpl/ClientInfo.java

@@ -18,7 +18,6 @@
  */
 package org.apache.accumulo.core.clientImpl;
 
-import java.net.URL;
 import java.nio.file.Path;
 import java.util.Optional;
 import java.util.Properties;
@@ -94,13 +93,6 @@ static ClientInfo from(Properties properties) {
     return new ClientInfoImpl(properties, Optional.empty());
   }
 
-  /**
-   * @return ClientInfo given URL path to client config file
-   */
-  static ClientInfo from(URL propertiesURL) {
-    return new ClientInfoImpl(ClientInfoImpl.toProperties(propertiesURL), Optional.empty());
-  }
-
   /**
    * @return ClientInfo given properties and token
    */

core/src/main/java/org/apache/accumulo/core/clientImpl/ClientTabletCache.java

@@ -39,7 +39,6 @@
 import org.apache.accumulo.core.dataImpl.KeyExtent;
 import org.apache.accumulo.core.util.Interner;
 import org.apache.accumulo.core.util.Timer;
-import org.apache.accumulo.core.util.UtilWaitThread;
 import org.apache.hadoop.io.Text;
 
 /**
@@ -116,17 +115,6 @@ public CachedTablet findTablet(ClientContext context, Text row, boolean skipRow,
     return findTablet(context, row, skipRow, locationNeed, 0, null);
   }
 
-  public CachedTablet findTabletWithRetry(ClientContext context, Text row, boolean skipRow,
-      LocationNeed locationNeed) throws AccumuloException, AccumuloSecurityException,
-      TableNotFoundException, InvalidTabletHostingRequestException {
-    var tl = findTablet(context, row, skipRow, locationNeed);
-    while (tl == null && locationNeed == LocationNeed.REQUIRED) {
-      UtilWaitThread.sleep(100);
-      tl = findTablet(context, row, skipRow, locationNeed);
-    }
-    return tl;
-  }
-
   public abstract <T extends Mutation> void binMutations(ClientContext context, List<T> mutations,
       Map<String,TabletServerMutations<T>> binnedMutations, List<T> failures)
       throws AccumuloException, AccumuloSecurityException, TableNotFoundException,

core/src/main/java/org/apache/accumulo/core/clientImpl/ClientTabletCacheImpl.java

@@ -611,11 +611,6 @@ public long getTabletHostingRequestCount() {
     return tabletHostingRequestCount.get();
   }
 
-  @VisibleForTesting
-  public void resetTabletHostingRequestCount() {
-    tabletHostingRequestCount.set(0);
-  }
-
   @VisibleForTesting
   public void enableTabletHostingRequests(boolean enabled) {
     HOSTING_ENABLED.set(enabled);

core/src/main/java/org/apache/accumulo/core/clientImpl/DelegationTokenImpl.java

@@ -30,15 +30,9 @@
 import org.apache.accumulo.core.client.security.tokens.PasswordToken;
 import org.apache.accumulo.core.securityImpl.thrift.TAuthenticationTokenIdentifier;
 import org.apache.hadoop.io.Text;
-import org.apache.hadoop.security.Credentials;
-import org.apache.hadoop.security.UserGroupInformation;
 import org.apache.hadoop.security.token.Token;
-import org.apache.hadoop.security.token.TokenIdentifier;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class DelegationTokenImpl extends PasswordToken implements DelegationToken {
-  private static final Logger log = LoggerFactory.getLogger(DelegationTokenImpl.class);
 
   public static final String SERVICE_NAME = "AccumuloDelegationToken";
 
@@ -56,41 +50,6 @@ public DelegationTokenImpl(byte[] delegationTokenPassword,
     this.identifier = identifier;
   }
 
-  public DelegationTokenImpl(String instanceID, UserGroupInformation user,
-      AuthenticationTokenIdentifier identifier) {
-    requireNonNull(instanceID);
-    requireNonNull(user);
-    requireNonNull(identifier);
-
-    Credentials creds = user.getCredentials();
-    Token<? extends TokenIdentifier> token =
-        creds.getToken(new Text(SERVICE_NAME + "-" + instanceID));
-    if (token == null) {
-      throw new IllegalArgumentException(
-          "Did not find Accumulo delegation token in provided UserGroupInformation");
-    }
-    setPasswordFromToken(token);
-    this.identifier = identifier;
-  }
-
-  public DelegationTokenImpl(Token<? extends TokenIdentifier> token,
-      AuthenticationTokenIdentifier identifier) {
-    requireNonNull(token);
-    requireNonNull(identifier);
-    setPasswordFromToken(token);
-    this.identifier = identifier;
-  }
-
-  private void setPasswordFromToken(Token<? extends TokenIdentifier> token) {
-    if (!AuthenticationTokenIdentifier.TOKEN_KIND.equals(token.getKind())) {
-      String msg = "Expected an AuthenticationTokenIdentifier but got a " + token.getKind();
-      log.error(msg);
-      throw new IllegalArgumentException(msg);
-    }
-
-    setPassword(token.getPassword());
-  }
-
   /**
    * The identifier for this token, may be null.
    */

core/src/main/java/org/apache/accumulo/core/clientImpl/TabletAvailabilityUtil.java

@@ -19,27 +19,10 @@
 package org.apache.accumulo.core.clientImpl;
 
 import org.apache.accumulo.core.client.admin.TabletAvailability;
-import org.apache.accumulo.core.clientImpl.thrift.TTabletAvailability;
 import org.apache.accumulo.core.data.Value;
 
 public class TabletAvailabilityUtil {
 
-  public static TabletAvailability fromThrift(TTabletAvailability tAvailability) {
-    return switch (tAvailability) {
-      case HOSTED -> TabletAvailability.HOSTED;
-      case UNHOSTED -> TabletAvailability.UNHOSTED;
-      case ONDEMAND -> TabletAvailability.ONDEMAND;
-    };
-  }
-
-  public static TTabletAvailability toThrift(TabletAvailability tabletAvailability) {
-    return switch (tabletAvailability) {
-      case HOSTED -> TTabletAvailability.HOSTED;
-      case UNHOSTED -> TTabletAvailability.UNHOSTED;
-      case ONDEMAND -> TTabletAvailability.ONDEMAND;
-    };
-  }
-
   public static TabletAvailability fromValue(Value value) {
     return switch (value.toString()) {
       case "HOSTED" -> TabletAvailability.HOSTED;

core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReaderIterator.java

@@ -843,15 +843,6 @@ public long getTimeOut() {
     }
   }
 
-  public static void doLookup(ClientContext context, String server,
-      Map<KeyExtent,List<Range>> requested, Map<KeyExtent,List<Range>> failures,
-      Map<KeyExtent,List<Range>> unscanned, ResultReceiver receiver, List<Column> columns,
-      ScannerOptions options, Authorizations authorizations)
-      throws IOException, AccumuloSecurityException, AccumuloServerException {
-    doLookup(context, server, requested, failures, unscanned, receiver, columns, options,
-        authorizations, new TimeoutTracker(Long.MAX_VALUE), 0L);
-  }
-
   static void doLookup(ClientContext context, String server, Map<KeyExtent,List<Range>> requested,
       Map<KeyExtent,List<Range>> failures, Map<KeyExtent,List<Range>> unscanned,
       ResultReceiver receiver, List<Column> columns, ScannerOptions options,