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();