You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2022/09/30 16:23:36 UTC

[accumulo] branch main updated: Fix trivial TableDiskUsage warnings

This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new 359b9f882c Fix trivial TableDiskUsage warnings
359b9f882c is described below

commit 359b9f882c4e7aba1cb886ecb0b870e58eeb1695
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Fri Sep 30 12:12:39 2022 -0400

    Fix trivial TableDiskUsage warnings
    
    Fix trivial warnings introduced by #2900 in TableDiskUsage
    
    * Remove unused variable assignment for volume manager
    * Fix improperly referenced inner class name in javadoc using fully
      qualified class name (maybe pre-existing problem)
---
 .../src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/server/base/src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java b/server/base/src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java
index 0046caa121..0c255f9e0d 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/util/TableDiskUsage.java
@@ -52,7 +52,6 @@ import org.apache.accumulo.core.security.Authorizations;
 import org.apache.accumulo.core.trace.TraceUtil;
 import org.apache.accumulo.core.util.NumUtil;
 import org.apache.accumulo.server.cli.ServerUtilOpts;
-import org.apache.accumulo.server.fs.VolumeManager;
 import org.apache.hadoop.fs.Path;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -65,7 +64,7 @@ import io.opentelemetry.context.Scope;
 /**
  * This utility class will scan the Accumulo Metadata table to compute the disk usage for a table or
  * table(s) by using the size value stored in columns that contain the column family
- * {@link MetadataSchema.TabletsSection.DataFileColumnFamily}.
+ * {@link org.apache.accumulo.core.metadata.schema.MetadataSchema.TabletsSection.DataFileColumnFamily}.
  *
  * This class will also track shared files to computed shared usage across all tables that are
  * provided as part of the Set of tables when getting disk usage.
@@ -342,7 +341,6 @@ public class TableDiskUsage {
     Span span = TraceUtil.startSpan(TableDiskUsage.class, "main");
     try (Scope scope = span.makeCurrent()) {
       try (AccumuloClient client = Accumulo.newClient().from(opts.getClientProps()).build()) {
-        VolumeManager fs = opts.getServerContext().getVolumeManager();
         org.apache.accumulo.server.util.TableDiskUsage.printDiskUsage(opts.tables, client, false);
       } finally {
         span.end();