mikewalch closed pull request #847: Remove empty comments
URL: https://github.com/apache/accumulo/pull/847
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git
a/core/src/main/java/org/apache/accumulo/core/iterators/user/RowEncodingIterator.java
b/core/src/main/java/org/apache/accumulo/core/iterators/user/RowEncodingIterator.java
index 79f4b6a45b..bac8849337 100644
---
a/core/src/main/java/org/apache/accumulo/core/iterators/user/RowEncodingIterator.java
+++
b/core/src/main/java/org/apache/accumulo/core/iterators/user/RowEncodingIterator.java
@@ -37,7 +37,6 @@
import org.apache.hadoop.io.Text;
/**
- *
* The RowEncodingIterator is designed to provide row-isolation so that
queries see mutations as
* atomic. It does so by encapsulating an entire row of key/value pairs into a
single key/value
* pair, which is returned through the client as an atomic operation. This is
an abstract class,
diff --git
a/core/src/main/java/org/apache/accumulo/core/spi/cache/BlockCacheManager.java
b/core/src/main/java/org/apache/accumulo/core/spi/cache/BlockCacheManager.java
index 89a17437a1..42c5635985 100644
---
a/core/src/main/java/org/apache/accumulo/core/spi/cache/BlockCacheManager.java
+++
b/core/src/main/java/org/apache/accumulo/core/spi/cache/BlockCacheManager.java
@@ -23,7 +23,6 @@
import org.apache.accumulo.core.conf.Property;
/**
- *
* @since 2.0.0
*/
public abstract class BlockCacheManager {
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/ActiveAssignmentRunnable.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/ActiveAssignmentRunnable.java
index 30b012ce19..e7c45192cb 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/ActiveAssignmentRunnable.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/ActiveAssignmentRunnable.java
@@ -24,9 +24,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class ActiveAssignmentRunnable implements Runnable {
private static final Logger log =
LoggerFactory.getLogger(ActiveAssignmentRunnable.class);
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/ConditionalMutationSet.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/ConditionalMutationSet.java
index f9e4852e1a..cd852c7c2f 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/ConditionalMutationSet.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/ConditionalMutationSet.java
@@ -27,9 +27,6 @@
import org.apache.accumulo.tserver.data.ServerConditionalMutation;
import org.apache.hadoop.io.WritableComparator;
-/**
- *
- */
public class ConditionalMutationSet {
interface DeferFilter {
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/MemValue.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/MemValue.java
index 67f379d995..94f4e6ce06 100644
--- a/server/tserver/src/main/java/org/apache/accumulo/tserver/MemValue.java
+++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/MemValue.java
@@ -18,9 +18,6 @@
import org.apache.accumulo.core.data.Value;
-/**
- *
- */
public class MemValue {
Value value;
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/RowLocks.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/RowLocks.java
index c0d8926c3d..fed66fb523 100644
--- a/server/tserver/src/main/java/org/apache/accumulo/tserver/RowLocks.java
+++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/RowLocks.java
@@ -29,9 +29,6 @@
import org.apache.accumulo.tserver.ConditionalMutationSet.DeferFilter;
import org.apache.accumulo.tserver.data.ServerConditionalMutation;
-/**
- *
- */
class RowLocks {
private Map<ByteSequence,RowLock> rowLocks = new HashMap<>();
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategy.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategy.java
index ccfb6ae3ef..0077964e81 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategy.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategy.java
@@ -25,9 +25,6 @@
import org.apache.accumulo.core.metadata.schema.DataFileValue;
import org.apache.accumulo.server.fs.FileRef;
-/**
- *
- */
public class SizeLimitCompactionStrategy extends DefaultCompactionStrategy {
public static final String SIZE_LIMIT_OPT = "sizeLimit";
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/data/ServerConditionalMutation.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/data/ServerConditionalMutation.java
index ab0846febb..01e127e6fa 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/data/ServerConditionalMutation.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/data/ServerConditionalMutation.java
@@ -23,9 +23,6 @@
import org.apache.accumulo.core.dataImpl.thrift.TConditionalMutation;
import org.apache.accumulo.server.data.ServerMutation;
-/**
- *
- */
public class ServerConditionalMutation extends ServerMutation {
public static class TCMTranslator
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/log/LogSorter.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/log/LogSorter.java
index 51379e10a6..5888b5c96d 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/log/LogSorter.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/log/LogSorter.java
@@ -51,9 +51,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class LogSorter {
private static final Logger log = LoggerFactory.getLogger(LogSorter.class);
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMetrics.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMetrics.java
index c0920396a8..f3f6b9dc96 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMetrics.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMetrics.java
@@ -28,9 +28,6 @@
import org.apache.hadoop.metrics2.lib.MetricsRegistry;
import org.apache.hadoop.metrics2.lib.MutableGaugeLong;
-/**
- *
- */
public class Metrics2TabletServerMetrics
implements Metrics, MetricsSource, TabletServerMetricsKeys {
public static final String NAME = TSERVER_NAME + ",sub=General",
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMinCMetrics.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMinCMetrics.java
index 6dc978e121..1427b7ce99 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMinCMetrics.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerMinCMetrics.java
@@ -27,9 +27,6 @@
import org.apache.hadoop.metrics2.lib.MetricsRegistry;
import org.apache.hadoop.metrics2.lib.MutableStat;
-/**
- *
- */
public class Metrics2TabletServerMinCMetrics
implements Metrics, MetricsSource, TabletServerMinCMetricsKeys {
public static final String NAME = TSERVER_NAME + ",sub=MinorCompactions",
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerScanMetrics.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerScanMetrics.java
index 0f44ca5647..294960ba7f 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerScanMetrics.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerScanMetrics.java
@@ -27,9 +27,6 @@
import org.apache.hadoop.metrics2.lib.MetricsRegistry;
import org.apache.hadoop.metrics2.lib.MutableStat;
-/**
- *
- */
public class Metrics2TabletServerScanMetrics
implements Metrics, MetricsSource, TabletServerScanMetricsKeys {
public static final String NAME = TSERVER_NAME + ",sub=Scans",
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerUpdateMetrics.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerUpdateMetrics.java
index 2022de4cd0..b21e05086a 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerUpdateMetrics.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/metrics/Metrics2TabletServerUpdateMetrics.java
@@ -28,9 +28,6 @@
import org.apache.hadoop.metrics2.lib.MutableCounterLong;
import org.apache.hadoop.metrics2.lib.MutableStat;
-/**
- *
- */
public class Metrics2TabletServerUpdateMetrics
implements Metrics, MetricsSource, TabletServerUpdateMetricsKeys {
public static final String NAME = TSERVER_NAME + ",sub=Updates",
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/replication/ReplicationServicerHandler.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/replication/ReplicationServicerHandler.java
index 0abe464d2b..aa887ca4b0 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/replication/ReplicationServicerHandler.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/replication/ReplicationServicerHandler.java
@@ -37,9 +37,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class ReplicationServicerHandler implements Iface {
private static final Logger log =
LoggerFactory.getLogger(ReplicationServicerHandler.class);
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactionWatcher.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactionWatcher.java
index 7382de4f63..769facd97e 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactionWatcher.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactionWatcher.java
@@ -28,9 +28,6 @@
import org.apache.accumulo.server.util.time.SimpleTimer;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class CompactionWatcher implements Runnable {
private final Map<List<Long>,ObservedCompactionInfo> observedCompactions =
new HashMap<>();
private final AccumuloConfiguration config;
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/RootFiles.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/RootFiles.java
index 32fb00901a..eb3d218955 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/RootFiles.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/RootFiles.java
@@ -32,9 +32,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-/**
- *
- */
public class RootFiles {
private static final Logger log = LoggerFactory.getLogger(RootFiles.class);
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
index a769061747..f1ecbb44f3 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
@@ -163,9 +163,7 @@
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
/**
- *
* Provide access to a single row range in a living TabletServer.
- *
*/
public class Tablet implements TabletCommitter {
private static final Logger log = LoggerFactory.getLogger(Tablet.class);
diff --git
a/server/tserver/src/test/java/org/apache/accumulo/tserver/CountingIteratorTest.java
b/server/tserver/src/test/java/org/apache/accumulo/tserver/CountingIteratorTest.java
index d4beebe140..6f60b89412 100644
---
a/server/tserver/src/test/java/org/apache/accumulo/tserver/CountingIteratorTest.java
+++
b/server/tserver/src/test/java/org/apache/accumulo/tserver/CountingIteratorTest.java
@@ -30,10 +30,8 @@
import org.apache.accumulo.tserver.tablet.CountingIterator;
import org.junit.Test;
-/**
- *
- */
public class CountingIteratorTest {
+
@Test
public void testDeepCopyCount() throws IOException {
TreeMap<Key,Value> tm = new TreeMap<>();
diff --git
a/server/tserver/src/test/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategyTest.java
b/server/tserver/src/test/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategyTest.java
index ded5837db8..e494426d7b 100644
---
a/server/tserver/src/test/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategyTest.java
+++
b/server/tserver/src/test/java/org/apache/accumulo/tserver/compaction/SizeLimitCompactionStrategyTest.java
@@ -33,10 +33,8 @@
import org.apache.accumulo.server.fs.FileRef;
import org.junit.Test;
-/**
- *
- */
public class SizeLimitCompactionStrategyTest {
+
private Map<FileRef,DataFileValue> nfl(String... sa) {
HashMap<FileRef,DataFileValue> ret = new HashMap<>();
diff --git
a/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/RootFilesTest.java
b/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/RootFilesTest.java
index d7aac43cef..be4cabe5a4 100644
---
a/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/RootFilesTest.java
+++
b/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/RootFilesTest.java
@@ -38,9 +38,6 @@
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
-/**
- *
- */
public class RootFilesTest {
@Rule
diff --git
a/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/TabletTest.java
b/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/TabletTest.java
index cd69d4c086..ea9c9ff541 100644
---
a/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/TabletTest.java
+++
b/server/tserver/src/test/java/org/apache/accumulo/tserver/tablet/TabletTest.java
@@ -34,9 +34,6 @@
import org.easymock.EasyMock;
import org.junit.Test;
-/**
- *
- */
public class TabletTest {
@Test
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
