HBASE-27220 Apply the spotless format change in HBASE-27208 to our code base

Signed-off-by: Andrew Purtell <apurtell@apache.org>

hbase-asyncfs/src/main/java/org/apache/hadoop/hbase/io/asyncfs/AsyncFSOutput.java

@@ -89,8 +89,6 @@ public interface AsyncFSOutput extends Closeable {
   @Override
   void close() throws IOException;
 
-  /**
+  /** Returns byteSize success synced to underlying filesystem. */
-   * @return byteSize success synced to underlying filesystem.
-   */
   long getSyncedLength();
 }

hbase-asyncfs/src/main/java/org/apache/hadoop/hbase/util/RecoverLeaseFSUtils.java

@@ -186,9 +186,7 @@ public final class RecoverLeaseFSUtils {
     return recovered;
   }
 
-  /**
+  /** Returns Detail to append to any log message around lease recovering. */
-   * @return Detail to append to any log message around lease recovering.
-   */
   private static String getLogMessageDetail(final int nbAttempt, final Path p,
     final long startWaiting) {
     return "attempt=" + nbAttempt + " on file=" + p + " after "

hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterId.java

@@ -45,9 +45,7 @@ public class ClusterId {
     this.id = uuid;
   }
 
-  /**
+  /** Returns The clusterid serialized using pb w/ pb magic prefix */
-   * @return The clusterid serialized using pb w/ pb magic prefix
-   */
   public byte[] toByteArray() {
     return ProtobufUtil.prependPBMagic(convert().toByteArray());
   }
@@ -74,9 +72,7 @@ public class ClusterId {
     }
   }
 
-  /**
+  /** Returns A pb instance to represent this instance. */
-   * @return A pb instance to represent this instance.
-   */
   public ClusterIdProtos.ClusterId convert() {
     ClusterIdProtos.ClusterId.Builder builder = ClusterIdProtos.ClusterId.newBuilder();
     return builder.setClusterId(this.id).build();

hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterMetrics.java

@@ -69,33 +69,23 @@ import org.apache.yetus.audience.InterfaceAudience;
 @InterfaceAudience.Public
 public interface ClusterMetrics {
 
-  /**
+  /** Returns the HBase version string as reported by the HMaster */
-   * @return the HBase version string as reported by the HMaster
-   */
   @Nullable
   String getHBaseVersion();
 
-  /**
+  /** Returns the names of region servers on the dead list */
-   * @return the names of region servers on the dead list
-   */
   List<ServerName> getDeadServerNames();
 
-  /**
+  /** Returns the names of region servers on the live list */
-   * @return the names of region servers on the live list
-   */
   Map<ServerName, ServerMetrics> getLiveServerMetrics();
 
-  /**
+  /** Returns the number of regions deployed on the cluster */
-   * @return the number of regions deployed on the cluster
-   */
   default int getRegionCount() {
     return getLiveServerMetrics().entrySet().stream()
       .mapToInt(v -> v.getValue().getRegionMetrics().size()).sum();
   }
 
-  /**
+  /** Returns the number of requests since last report */
-   * @return the number of requests since last report
-   */
   default long getRequestCount() {
     return getLiveServerMetrics().entrySet().stream()
       .flatMap(v -> v.getValue().getRegionMetrics().values().stream())
@@ -109,9 +99,7 @@ public interface ClusterMetrics {
   @Nullable
   ServerName getMasterName();
 
-  /**
+  /** Returns the names of backup masters */
-   * @return the names of backup masters
-   */
   List<ServerName> getBackupMasterNames();
 
   @InterfaceAudience.Private
@@ -142,9 +130,7 @@ public interface ClusterMetrics {
 
   List<ServerName> getServersName();
 
-  /**
+  /** Returns the average cluster load */
-   * @return the average cluster load
-   */
   default double getAverageLoad() {
     int serverSize = getLiveServerMetrics().size();
     if (serverSize == 0) {

hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java

@@ -107,9 +107,7 @@ public class ClusterStatus implements ClusterMetrics {
     this.metrics = metrics;
   }
 
-  /**
+  /** Returns the names of region servers on the dead list */
-   * @return the names of region servers on the dead list
-   */
   @Override
   public List<ServerName> getDeadServerNames() {
     return metrics.getDeadServerNames();
@@ -187,9 +185,7 @@ public class ClusterStatus implements ClusterMetrics {
     return metrics.getRegionStatesInTransition();
   }
 
-  /**
+  /** Returns the HBase version string as reported by the HMaster */
-   * @return the HBase version string as reported by the HMaster
-   */
   public String getHBaseVersion() {
     return metrics.getHBaseVersion();
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/CoprocessorEnvironment.java

@@ -28,19 +28,19 @@ import org.apache.yetus.audience.InterfaceStability;
 @InterfaceStability.Evolving
 public interface CoprocessorEnvironment<C extends Coprocessor> {
 
-  /** @return the Coprocessor interface version */
+  /** Returns the Coprocessor interface version */
   int getVersion();
 
-  /** @return the HBase version as a string (e.g. "0.21.0") */
+  /** Returns the HBase version as a string (e.g. "0.21.0") */
   String getHBaseVersion();
 
-  /** @return the loaded coprocessor instance */
+  /** Returns the loaded coprocessor instance */
   C getInstance();
 
-  /** @return the priority assigned to the loaded coprocessor */
+  /** Returns the priority assigned to the loaded coprocessor */
   int getPriority();
 
-  /** @return the load sequence number */
+  /** Returns the load sequence number */
   int getLoadSequence();
 
   /**
@@ -49,8 +49,6 @@ public interface CoprocessorEnvironment<C extends Coprocessor> {
    */
   Configuration getConfiguration();
 
-  /**
+  /** Returns the classloader for the loaded coprocessor instance */
-   * @return the classloader for the loaded coprocessor instance
-   */
   ClassLoader getClassLoader();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/HBaseServerException.java

@@ -63,9 +63,7 @@ public class HBaseServerException extends HBaseIOException {
     this.serverOverloaded = serverOverloaded;
   }
 
-  /**
+  /** Returns True if server was considered overloaded when exception was thrown */
-   * @return True if server was considered overloaded when exception was thrown
-   */
   public boolean isServerOverloaded() {
     return serverOverloaded;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java

@@ -193,17 +193,13 @@ public class HColumnDescriptor implements ColumnFamilyDescriptor, Comparable<HCo
     return ColumnFamilyDescriptorBuilder.isLegalColumnFamilyName(b);
   }
 
-  /**
+  /** Returns Name of this column family */
-   * @return Name of this column family
-   */
   @Override
   public byte[] getName() {
     return delegatee.getName();
   }
 
-  /**
+  /** Returns The name string of this column family */
-   * @return The name string of this column family
-   */
   @Override
   public String getNameAsString() {
     return delegatee.getNameAsString();
@@ -650,9 +646,7 @@ public class HColumnDescriptor implements ColumnFamilyDescriptor, Comparable<HCo
     return delegatee.toString();
   }
 
-  /**
+  /** Returns Column family descriptor with only the customized attributes. */
-   * @return Column family descriptor with only the customized attributes.
-   */
   @Override
   public String toStringCustomizedValues() {
     return delegatee.toStringCustomizedValues();

hbase-client/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java

@@ -104,7 +104,7 @@ public class HRegionInfo implements RegionInfo {
   }
 
   /**
-   * @return Return a short, printable name for this region (usually encoded name) for us logging.
+   * Returns Return a short, printable name for this region (usually encoded name) for us logging.
    */
   @Override
   public String getShortNameToLog() {
@@ -425,7 +425,7 @@ public class HRegionInfo implements RegionInfo {
     return RegionInfo.isEncodedRegionName(regionName);
   }
 
-  /** @return the regionId */
+  /** Returns the regionId */
   @Override
   public long getRegionId() {
     return regionId;
@@ -440,9 +440,7 @@ public class HRegionInfo implements RegionInfo {
     return regionName;
   }
 
-  /**
+  /** Returns Region name as a String for use in logging, etc. */
-   * @return Region name as a String for use in logging, etc.
-   */
   @Override
   public String getRegionNameAsString() {
     if (RegionInfo.hasEncodedName(this.regionName)) {
@@ -456,9 +454,7 @@ public class HRegionInfo implements RegionInfo {
     return Bytes.toStringBinary(this.regionName) + "." + this.getEncodedName();
   }
 
-  /**
+  /** Returns the encoded region name */
-   * @return the encoded region name
-   */
   @Override
   public synchronized String getEncodedName() {
     if (this.encodedName == null) {
@@ -475,17 +471,13 @@ public class HRegionInfo implements RegionInfo {
     return this.encodedNameAsBytes;
   }
 
-  /**
+  /** Returns the startKey */
-   * @return the startKey
-   */
   @Override
   public byte[] getStartKey() {
     return startKey;
   }
 
-  /**
+  /** Returns the endKey */
-   * @return the endKey
-   */
   @Override
   public byte[] getEndKey() {
     return endKey;
@@ -524,40 +516,30 @@ public class HRegionInfo implements RegionInfo {
     return firstKeyInRange && lastKeyInRange;
   }
 
-  /**
+  /** Returns true if the given row falls in this region. */
-   * @return true if the given row falls in this region.
-   */
   @Override
   public boolean containsRow(byte[] row) {
     return Bytes.compareTo(row, startKey) >= 0
       && (Bytes.compareTo(row, endKey) < 0 || Bytes.equals(endKey, HConstants.EMPTY_BYTE_ARRAY));
   }
 
-  /**
+  /** Returns true if this region is from hbase:meta */
-   * @return true if this region is from hbase:meta
-   */
   public boolean isMetaTable() {
     return isMetaRegion();
   }
 
-  /**
+  /** Returns true if this region is a meta region */
-   * @return true if this region is a meta region
-   */
   @Override
   public boolean isMetaRegion() {
     return tableName.equals(HRegionInfo.FIRST_META_REGIONINFO.getTable());
   }
 
-  /**
+  /** Returns true if this region is from a system table */
-   * @return true if this region is from a system table
-   */
   public boolean isSystemTable() {
     return tableName.isSystemTable();
   }
 
-  /**
+  /** Returns true if has been split and has daughters. */
-   * @return true if has been split and has daughters.
-   */
   @Override
   public boolean isSplit() {
     return this.split;
@@ -570,9 +552,7 @@ public class HRegionInfo implements RegionInfo {
     this.split = split;
   }
 
-  /**
+  /** Returns true if this region is offline. */
-   * @return true if this region is offline.
-   */
   @Override
   public boolean isOffline() {
     return this.offLine;
@@ -587,9 +567,7 @@ public class HRegionInfo implements RegionInfo {
     this.offLine = offLine;
   }
 
-  /**
+  /** Returns true if this is a split parent region. */
-   * @return true if this is a split parent region.
-   */
   @Override
   public boolean isSplitParent() {
     if (!isSplit()) return false;

hbase-client/src/main/java/org/apache/hadoop/hbase/HTableDescriptor.java

@@ -160,9 +160,7 @@ public class HTableDescriptor implements TableDescriptor, Comparable<HTableDescr
     return delegatee.isMetaTable();
   }
 
-  /**
+  /** Returns Getter for fetching an unmodifiable map. */
-   * @return Getter for fetching an unmodifiable map.
-   */
   @Override
   public Map<Bytes, Bytes> getValues() {
     return delegatee.getValues();
@@ -525,9 +523,7 @@ public class HTableDescriptor implements TableDescriptor, Comparable<HTableDescr
     return delegatee.toStringCustomizedValues();
   }
 
-  /**
+  /** Returns map of all table attributes formatted into string. */
-   * @return map of all table attributes formatted into string.
-   */
   public String toStringTableAttributes() {
     return delegatee.toStringTableAttributes();
   }
@@ -611,9 +607,7 @@ public class HTableDescriptor implements TableDescriptor, Comparable<HTableDescr
     return hasRegionMemStoreReplication();
   }
 
-  /**
+  /** Returns true if the read-replicas memstore replication is enabled. */
-   * @return true if the read-replicas memstore replication is enabled.
-   */
   @Override
   public boolean hasRegionMemStoreReplication() {
     return delegatee.hasRegionMemStoreReplication();

hbase-client/src/main/java/org/apache/hadoop/hbase/MetaTableAccessor.java

@@ -304,9 +304,7 @@ public class MetaTableAccessor {
       regionInfo.getReplicaId());
   }
 
-  /**
+  /** Returns Return the {@link HConstants#CATALOG_FAMILY} row from hbase:meta. */
-   * @return Return the {@link HConstants#CATALOG_FAMILY} row from hbase:meta.
-   */
   public static Result getCatalogFamilyRow(Connection connection, RegionInfo ri)
     throws IOException {
     Get get = new Get(getMetaKeyForRegion(ri));
@@ -433,9 +431,7 @@ public class MetaTableAccessor {
     return false;
   }
 
-  /**
+  /** Returns True if the column in <code>cell</code> matches the regex 'info:merge.*'. */
-   * @return True if the column in <code>cell</code> matches the regex 'info:merge.*'.
-   */
   private static boolean isMergeQualifierPrefix(Cell cell) {
     // Check to see if has family and that qualifier starts with the merge qualifier 'merge'
     return CellUtil.matchingFamily(cell, HConstants.CATALOG_FAMILY)
@@ -825,9 +821,7 @@ public class MetaTableAccessor {
     }
   }
 
-  /**
+  /** Returns Get closest metatable region row to passed <code>row</code> */
-   * @return Get closest metatable region row to passed <code>row</code>
-   */
   @NonNull
   private static RegionInfo getClosestRegionInfo(Connection connection,
     @NonNull final TableName tableName, @NonNull final byte[] row) throws IOException {
@@ -1228,9 +1222,7 @@ public class MetaTableAccessor {
 
     abstract void add(Result r);
 
-    /**
+    /** Returns Collected results; wait till visits complete to collect all possible results */
-     * @return Collected results; wait till visits complete to collect all possible results
-     */
     List<T> getResults() {
       return this.results;
     }

hbase-client/src/main/java/org/apache/hadoop/hbase/RegionLoad.java

@@ -332,9 +332,7 @@ public class RegionLoad implements RegionMetrics {
     return (int) metrics.getUncompressedStoreFileSize().get(Size.Unit.KILOBYTE);
   }
 
-  /**
+  /** Returns the data locality of region in the regionserver. */
-   * @return the data locality of region in the regionserver.
-   */
   @Override
   public float getDataLocality() {
     return metrics.getDataLocality();
@@ -355,9 +353,7 @@ public class RegionLoad implements RegionMetrics {
     return metrics.getLastMajorCompactionTimestamp();
   }
 
-  /**
+  /** Returns the reference count for the stores of this region */
-   * @return the reference count for the stores of this region
-   */
   public int getStoreRefCount() {
     return metrics.getStoreRefCount();
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/RegionMetrics.java

@@ -28,58 +28,38 @@ import org.apache.yetus.audience.InterfaceAudience;
 @InterfaceAudience.Public
 public interface RegionMetrics {
 
-  /**
+  /** Returns the region name */
-   * @return the region name
-   */
   byte[] getRegionName();
 
-  /**
+  /** Returns the number of stores */
-   * @return the number of stores
-   */
   int getStoreCount();
 
-  /**
+  /** Returns the number of storefiles */
-   * @return the number of storefiles
-   */
   int getStoreFileCount();
 
-  /**
+  /** Returns the total size of the storefiles */
-   * @return the total size of the storefiles
-   */
   Size getStoreFileSize();
 
-  /**
+  /** Returns the memstore size */
-   * @return the memstore size
-   */
   Size getMemStoreSize();
 
-  /**
+  /** Returns the number of read requests made to region */
-   * @return the number of read requests made to region
-   */
   long getReadRequestCount();
 
-  /**
+  /** Returns the number of write requests made to region */
-   * @return the number of write requests made to region
-   */
   long getWriteRequestCount();
 
-  /**
+  /** Returns the number of write requests and read requests made to region */
-   * @return the number of write requests and read requests made to region
-   */
   default long getRequestCount() {
     return getReadRequestCount() + getWriteRequestCount();
   }
 
-  /**
+  /** Returns the region name as a string */
-   * @return the region name as a string
-   */
   default String getNameAsString() {
     return Bytes.toStringBinary(getRegionName());
   }
 
-  /**
+  /** Returns the number of filtered read requests made to region */
-   * @return the number of filtered read requests made to region
-   */
   long getFilteredReadRequestCount();
 
   /**
@@ -90,29 +70,19 @@ public interface RegionMetrics {
    */
   Size getStoreFileIndexSize();
 
-  /**
+  /** Returns The current total size of root-level indexes for the region */
-   * @return The current total size of root-level indexes for the region
-   */
   Size getStoreFileRootLevelIndexSize();
 
-  /**
+  /** Returns The total size of all index blocks, not just the root level */
-   * @return The total size of all index blocks, not just the root level
-   */
   Size getStoreFileUncompressedDataIndexSize();
 
-  /**
+  /** Returns The total size of all Bloom filter blocks, not just loaded into the block cache */
-   * @return The total size of all Bloom filter blocks, not just loaded into the block cache
-   */
   Size getBloomFilterSize();
 
-  /**
+  /** Returns the total number of cells in current compaction */
-   * @return the total number of cells in current compaction
-   */
   long getCompactingCellCount();
 
-  /**
+  /** Returns the number of already compacted kvs in current compaction */
-   * @return the number of already compacted kvs in current compaction
-   */
   long getCompactedCellCount();
 
   /**
@@ -121,29 +91,19 @@ public interface RegionMetrics {
    */
   long getCompletedSequenceId();
 
-  /**
+  /** Returns completed sequence id per store. */
-   * @return completed sequence id per store.
-   */
   Map<byte[], Long> getStoreSequenceId();
 
-  /**
+  /** Returns the uncompressed size of the storefiles */
-   * @return the uncompressed size of the storefiles
-   */
   Size getUncompressedStoreFileSize();
 
-  /**
+  /** Returns the data locality of region in the regionserver. */
-   * @return the data locality of region in the regionserver.
-   */
   float getDataLocality();
 
-  /**
+  /** Returns the timestamp of the oldest hfile for any store of this region. */
-   * @return the timestamp of the oldest hfile for any store of this region.
-   */
   long getLastMajorCompactionTimestamp();
 
-  /**
+  /** Returns the reference count for the stores of this region */
-   * @return the reference count for the stores of this region
-   */
   int getStoreRefCount();
 
   /**
@@ -158,9 +118,7 @@ public interface RegionMetrics {
    */
   float getDataLocalityForSsd();
 
-  /**
+  /** Returns the data at local weight of this region in the regionserver */
-   * @return the data at local weight of this region in the regionserver
-   */
   long getBlocksLocalWeight();
 
   /**
@@ -169,13 +127,9 @@ public interface RegionMetrics {
    */
   long getBlocksLocalWithSsdWeight();
 
-  /**
+  /** Returns the block total weight of this region */
-   * @return the block total weight of this region
-   */
   long getBlocksTotalWeight();
 
-  /**
+  /** Returns the compaction state of this region */
-   * @return the compaction state of this region
-   */
   CompactionState getCompactionState();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/ServerMetrics.java

@@ -33,38 +33,26 @@ public interface ServerMetrics {
 
   ServerName getServerName();
 
-  /**
+  /** Returns the version number of a regionserver. */
-   * @return the version number of a regionserver.
-   */
   default int getVersionNumber() {
     return 0;
   }
 
-  /**
+  /** Returns the string type version of a regionserver. */
-   * @return the string type version of a regionserver.
-   */
   default String getVersion() {
     return "0.0.0";
   }
 
-  /**
+  /** Returns the number of requests per second. */
-   * @return the number of requests per second.
-   */
   long getRequestCountPerSecond();
 
-  /**
+  /** Returns total Number of requests from the start of the region server. */
-   * @return total Number of requests from the start of the region server.
-   */
   long getRequestCount();
 
-  /**
+  /** Returns the amount of used heap */
-   * @return the amount of used heap
-   */
   Size getUsedHeapSize();
 
-  /**
+  /** Returns the maximum allowable size of the heap */
-   * @return the maximum allowable size of the heap
-   */
   Size getMaxHeapSize();
 
   int getInfoServerPort();
@@ -87,14 +75,10 @@ public interface ServerMetrics {
   @Nullable
   ReplicationLoadSink getReplicationLoadSink();
 
-  /**
+  /** Returns region load metrics */
-   * @return region load metrics
-   */
   Map<byte[], RegionMetrics> getRegionMetrics();
 
-  /**
+  /** Returns metrics per user */
-   * @return metrics per user
-   */
   Map<byte[], UserMetrics> getUserMetrics();
 
   /**
@@ -103,14 +87,10 @@ public interface ServerMetrics {
    */
   Set<String> getCoprocessorNames();
 
-  /**
+  /** Returns the timestamp (server side) of generating this metrics */
-   * @return the timestamp (server side) of generating this metrics
-   */
   long getReportTimestamp();
 
-  /**
+  /** Returns the last timestamp (server side) of generating this metrics */
-   * @return the last timestamp (server side) of generating this metrics
-   */
   long getLastReportTimestamp();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/Size.java

@@ -69,9 +69,7 @@ public final class Size implements Comparable<Size> {
     this.unit = Preconditions.checkNotNull(unit);
   }
 
-  /**
+  /** Returns size unit */
-   * @return size unit
-   */
   public Unit getUnit() {
     return unit;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/UserMetrics.java

@@ -40,19 +40,13 @@ public interface UserMetrics {
     long getFilteredReadRequestsCount();
   }
 
-  /**
+  /** Returns the user name */
-   * @return the user name
-   */
   byte[] getUserName();
 
-  /**
+  /** Returns the number of read requests made by user */
-   * @return the number of read requests made by user
-   */
   long getReadRequestCount();
 
-  /**
+  /** Returns the number of write requests made by user */
-   * @return the number of write requests made by user
-   */
   long getWriteRequestCount();
 
   /**
@@ -63,20 +57,14 @@ public interface UserMetrics {
     return getReadRequestCount() + getWriteRequestCount();
   }
 
-  /**
+  /** Returns the user name as a string */
-   * @return the user name as a string
-   */
   default String getNameAsString() {
     return Bytes.toStringBinary(getUserName());
   }
 
-  /**
+  /** Returns metrics per client(hostname) */
-   * @return metrics per client(hostname)
-   */
   Map<String, ClientMetrics> getClientMetrics();
 
-  /**
+  /** Returns count of filtered read requests for a user */
-   * @return count of filtered read requests for a user
-   */
   long getFilteredReadRequests();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Admin.java

@@ -112,9 +112,7 @@ public interface Admin extends Abortable, Closeable {
   @Override
   boolean isAborted();
 
-  /**
+  /** Returns Connection used by this object. */
-   * @return Connection used by this object.
-   */
   Connection getConnection();
 
   /**
@@ -1710,9 +1708,7 @@ public interface Admin extends Abortable, Closeable {
   List<RegionMetrics> getRegionMetrics(ServerName serverName, TableName tableName)
     throws IOException;
 
-  /**
+  /** Returns Configuration used by the instance. */
-   * @return Configuration used by the instance.
-   */
   Configuration getConfiguration();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Append.java

@@ -90,9 +90,7 @@ public class Append extends Mutation {
     return this;
   }
 
-  /**
+  /** Returns current setting for returnResults */
-   * @return current setting for returnResults
-   */
   // This method makes public the superclasses's protected method.
   @Override
   public boolean isReturnResults() {

hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java

@@ -1051,34 +1051,24 @@ public interface AsyncAdmin {
   CompletableFuture<Void> recommissionRegionServer(ServerName server,
     List<byte[]> encodedRegionNames);
 
-  /**
+  /** Returns cluster status wrapped by {@link CompletableFuture} */
-   * @return cluster status wrapped by {@link CompletableFuture}
-   */
   CompletableFuture<ClusterMetrics> getClusterMetrics();
 
-  /**
+  /** Returns cluster status wrapped by {@link CompletableFuture} */
-   * @return cluster status wrapped by {@link CompletableFuture}
-   */
   CompletableFuture<ClusterMetrics> getClusterMetrics(EnumSet<Option> options);
 
-  /**
+  /** Returns current master server name wrapped by {@link CompletableFuture} */
-   * @return current master server name wrapped by {@link CompletableFuture}
-   */
   default CompletableFuture<ServerName> getMaster() {
     return getClusterMetrics(EnumSet.of(Option.MASTER)).thenApply(ClusterMetrics::getMasterName);
   }
 
-  /**
+  /** Returns current backup master list wrapped by {@link CompletableFuture} */
-   * @return current backup master list wrapped by {@link CompletableFuture}
-   */
   default CompletableFuture<Collection<ServerName>> getBackupMasters() {
     return getClusterMetrics(EnumSet.of(Option.BACKUP_MASTERS))
       .thenApply(ClusterMetrics::getBackupMasterNames);
   }
 
-  /**
+  /** Returns current live region servers list wrapped by {@link CompletableFuture} */
-   * @return current live region servers list wrapped by {@link CompletableFuture}
-   */
   default CompletableFuture<Collection<ServerName>> getRegionServers() {
     return getClusterMetrics(EnumSet.of(Option.SERVERS_NAME))
       .thenApply(ClusterMetrics::getServersName);
@@ -1110,9 +1100,7 @@ public interface AsyncAdmin {
     return future;
   }
 
-  /**
+  /** Returns a list of master coprocessors wrapped by {@link CompletableFuture} */
-   * @return a list of master coprocessors wrapped by {@link CompletableFuture}
-   */
   default CompletableFuture<List<String>> getMasterCoprocessorNames() {
     return getClusterMetrics(EnumSet.of(Option.MASTER_COPROCESSORS))
       .thenApply(ClusterMetrics::getMasterCoprocessorNames);

hbase-client/src/main/java/org/apache/hadoop/hbase/client/CheckAndMutate.java

@@ -250,9 +250,7 @@ public final class CheckAndMutate implements Row {
     this.action = action;
   }
 
-  /**
+  /** Returns the row */
-   * @return the row
-   */
   @Override
   public byte[] getRow() {
     return row;
@@ -281,58 +279,42 @@ public final class CheckAndMutate implements Row {
     return Bytes.hashCode(this.getRow());
   }
 
-  /**
+  /** Returns the family to check */
-   * @return the family to check
-   */
   public byte[] getFamily() {
     return family;
   }
 
-  /**
+  /** Returns the qualifier to check */
-   * @return the qualifier to check
-   */
   public byte[] getQualifier() {
     return qualifier;
   }
 
-  /**
+  /** Returns the comparison operator */
-   * @return the comparison operator
-   */
   public CompareOperator getCompareOp() {
     return op;
   }
 
-  /**
+  /** Returns the expected value */
-   * @return the expected value
-   */
   public byte[] getValue() {
     return value;
   }
 
-  /**
+  /** Returns the filter to check */
-   * @return the filter to check
-   */
   public Filter getFilter() {
     return filter;
   }
 
-  /**
+  /** Returns whether this has a filter or not */
-   * @return whether this has a filter or not
-   */
   public boolean hasFilter() {
     return filter != null;
   }
 
-  /**
+  /** Returns the time range to check */
-   * @return the time range to check
-   */
   public TimeRange getTimeRange() {
     return timeRange;
   }
 
-  /**
+  /** Returns the action done if check succeeds */
-   * @return the action done if check succeeds
-   */
   public Row getAction() {
     return action;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/CheckAndMutateResult.java

@@ -32,16 +32,12 @@ public class CheckAndMutateResult {
     this.result = result;
   }
 
-  /**
+  /** Returns Whether the CheckAndMutate operation is successful or not */
-   * @return Whether the CheckAndMutate operation is successful or not
-   */
   public boolean isSuccess() {
     return success;
   }
 
-  /**
+  /** Returns It is used only for CheckAndMutate operations with Increment/Append. Otherwise null */
-   * @return It is used only for CheckAndMutate operations with Increment/Append. Otherwise null
-   */
   public Result getResult() {
     return result;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClientIdGenerator.java

@@ -58,9 +58,7 @@ final class ClientIdGenerator {
     return id;
   }
 
-  /**
+  /** Returns PID of the current process, if it can be extracted from JVM name, or null. */
-   * @return PID of the current process, if it can be extracted from JVM name, or null.
-   */
   public static Long getPid() {
     String name = ManagementFactory.getRuntimeMXBean().getName();
     String[] nameParts = name.split("@");

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClusterConnection.java

@@ -246,13 +246,11 @@ public interface ClusterConnection extends Connection {
   void clearCaches(final ServerName sn);
 
   /**
-   * @return Nonce generator for this ClusterConnection; may be null if disabled in configuration.
+   * Returns Nonce generator for this ClusterConnection; may be null if disabled in configuration.
    */
   NonceGenerator getNonceGenerator();
 
-  /**
+  /** Returns Default AsyncProcess associated with this connection. */
-   * @return Default AsyncProcess associated with this connection.
-   */
   AsyncProcess getAsyncProcess();
 
   /**
@@ -263,34 +261,22 @@ public interface ClusterConnection extends Connection {
    */
   RpcRetryingCallerFactory getNewRpcRetryingCallerFactory(Configuration conf);
 
-  /**
+  /** Returns Connection's RpcRetryingCallerFactory instance */
-   * @return Connection's RpcRetryingCallerFactory instance
-   */
   RpcRetryingCallerFactory getRpcRetryingCallerFactory();
 
-  /**
+  /** Returns Connection's RpcControllerFactory instance */
-   * @return Connection's RpcControllerFactory instance
-   */
   RpcControllerFactory getRpcControllerFactory();
 
-  /**
+  /** Returns a ConnectionConfiguration object holding parsed configuration values */
-   * @return a ConnectionConfiguration object holding parsed configuration values
-   */
   ConnectionConfiguration getConnectionConfiguration();
 
-  /**
+  /** Returns the current statistics tracker associated with this connection */
-   * @return the current statistics tracker associated with this connection
-   */
   ServerStatisticTracker getStatisticsTracker();
 
-  /**
+  /** Returns the configured client backoff policy */
-   * @return the configured client backoff policy
-   */
   ClientBackoffPolicy getBackoffPolicy();
 
-  /**
+  /** Returns the MetricsConnection instance associated with this connection. */
-   * @return the MetricsConnection instance associated with this connection.
-   */
   MetricsConnection getConnectionMetrics();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ColumnFamilyDescriptor.java

@@ -77,39 +77,25 @@ public interface ColumnFamilyDescriptor {
       return lcf.getConfiguration().hashCode() - rcf.getConfiguration().hashCode();
     };
 
-  /**
+  /** Returns The storefile/hfile blocksize for this column family. */
-   * @return The storefile/hfile blocksize for this column family.
-   */
   int getBlocksize();
 
-  /**
+  /** Returns bloom filter type used for new StoreFiles in ColumnFamily */
-   * @return bloom filter type used for new StoreFiles in ColumnFamily
-   */
   BloomType getBloomFilterType();
 
-  /**
+  /** Returns Compression type setting. */
-   * @return Compression type setting.
-   */
   Compression.Algorithm getCompactionCompressionType();
 
-  /**
+  /** Returns Compression type setting for major compactions. */
-   * @return Compression type setting for major compactions.
-   */
   Compression.Algorithm getMajorCompactionCompressionType();
 
-  /**
+  /** Returns Compression type setting for minor compactions. */
-   * @return Compression type setting for minor compactions.
-   */
   Compression.Algorithm getMinorCompactionCompressionType();
 
-  /**
+  /** Returns Compression type setting. */
-   * @return Compression type setting.
-   */
   Compression.Algorithm getCompressionType();
 
-  /**
+  /** Returns an unmodifiable map. */
-   * @return an unmodifiable map.
-   */
   Map<String, String> getConfiguration();
 
   /**
@@ -118,24 +104,16 @@ public interface ColumnFamilyDescriptor {
    */
   String getConfigurationValue(String key);
 
-  /**
+  /** Returns replication factor set for this CF */
-   * @return replication factor set for this CF
-   */
   short getDFSReplication();
 
-  /**
+  /** Returns the data block encoding algorithm used in block cache and optionally on disk */
-   * @return the data block encoding algorithm used in block cache and optionally on disk
-   */
   DataBlockEncoding getDataBlockEncoding();
 
-  /**
+  /** Returns Return the raw crypto key attribute for the family, or null if not set */
-   * @return Return the raw crypto key attribute for the family, or null if not set
-   */
   byte[] getEncryptionKey();
 
-  /**
+  /** Returns Return the encryption algorithm in use by this family */
-   * @return Return the encryption algorithm in use by this family
-   */
   String getEncryptionType();
 
   /**
@@ -144,19 +122,13 @@ public interface ColumnFamilyDescriptor {
    */
   MemoryCompactionPolicy getInMemoryCompaction();
 
-  /**
+  /** Returns return the KeepDeletedCells */
-   * @return return the KeepDeletedCells
-   */
   KeepDeletedCells getKeepDeletedCells();
 
-  /**
+  /** Returns maximum number of versions */
-   * @return maximum number of versions
-   */
   int getMaxVersions();
 
-  /**
+  /** Returns The minimum number of versions to keep. */
-   * @return The minimum number of versions to keep.
-   */
   int getMinVersions();
 
   /**
@@ -171,19 +143,13 @@ public interface ColumnFamilyDescriptor {
    */
   long getMobThreshold();
 
-  /**
+  /** Returns a copy of Name of this column family */
-   * @return a copy of Name of this column family
-   */
   byte[] getName();
 
-  /**
+  /** Returns Name of this column family */
-   * @return Name of this column family
-   */
   String getNameAsString();
 
-  /**
+  /** Returns the scope tag */
-   * @return the scope tag
-   */
   int getScope();
 
   /**
@@ -193,9 +159,7 @@ public interface ColumnFamilyDescriptor {
    */
   String getStoragePolicy();
 
-  /**
+  /** Returns Time-to-live of cell contents, in seconds. */
-   * @return Time-to-live of cell contents, in seconds.
-   */
   int getTimeToLive();
 
   /**
@@ -228,19 +192,13 @@ public interface ColumnFamilyDescriptor {
    */
   boolean isBlockCacheEnabled();
 
-  /**
+  /** Returns true if we should cache bloomfilter blocks on write */
-   * @return true if we should cache bloomfilter blocks on write
-   */
   boolean isCacheBloomsOnWrite();
 
-  /**
+  /** Returns true if we should cache data blocks on write */
-   * @return true if we should cache data blocks on write
-   */
   boolean isCacheDataOnWrite();
 
-  /**
+  /** Returns true if we should cache index blocks on write */
-   * @return true if we should cache index blocks on write
-   */
   boolean isCacheIndexesOnWrite();
 
   /**
@@ -249,9 +207,7 @@ public interface ColumnFamilyDescriptor {
    */
   boolean isCompressTags();
 
-  /**
+  /** Returns true if we should evict cached blocks from the blockcache on close */
-   * @return true if we should evict cached blocks from the blockcache on close
-   */
   boolean isEvictBlocksOnClose();
 
   /**
@@ -266,14 +222,10 @@ public interface ColumnFamilyDescriptor {
    */
   boolean isMobEnabled();
 
-  /**
+  /** Returns true if we should prefetch blocks into the blockcache on open */
-   * @return true if we should prefetch blocks into the blockcache on open
-   */
   boolean isPrefetchBlocksOnOpen();
 
-  /**
+  /** Returns Column family descriptor with only the customized attributes. */
-   * @return Column family descriptor with only the customized attributes.
-   */
   String toStringCustomizedValues();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Connection.java

@@ -58,9 +58,7 @@ public interface Connection extends Abortable, Closeable {
    * general methods.
    */
 
-  /**
+  /** Returns Configuration instance being used by this Connection instance. */
-   * @return Configuration instance being used by this Connection instance.
-   */
   Configuration getConfiguration();
 
   /**
@@ -173,9 +171,7 @@ public interface Connection extends Abortable, Closeable {
    */
   TableBuilder getTableBuilder(TableName tableName, ExecutorService pool);
 
-  /**
+  /** Returns the cluster ID unique to this HBase cluster. */
-   * @return the cluster ID unique to this HBase cluster.
-   */
   String getClusterId();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionRegistryFactory.java

@@ -32,9 +32,7 @@ final class ConnectionRegistryFactory {
   private ConnectionRegistryFactory() {
   }
 
-  /**
+  /** Returns The connection registry implementation to use. */
-   * @return The connection registry implementation to use.
-   */
   static ConnectionRegistry getRegistry(Configuration conf) {
     Class<? extends ConnectionRegistry> clazz =
       conf.getClass(CLIENT_CONNECTION_REGISTRY_IMPL_CONF_KEY, ZKConnectionRegistry.class,

hbase-client/src/main/java/org/apache/hadoop/hbase/client/CoprocessorDescriptor.java

@@ -28,23 +28,17 @@ import org.apache.yetus.audience.InterfaceAudience;
  */
 @InterfaceAudience.Public
 public interface CoprocessorDescriptor {
-  /**
+  /** Returns the name of the class or interface represented by this object. */
-   * @return the name of the class or interface represented by this object.
-   */
   String getClassName();
 
   /**
-   * @return Path of the jar file. If it's null, the class will be loaded from default classloader.
+   * Returns Path of the jar file. If it's null, the class will be loaded from default classloader.
    */
   Optional<String> getJarPath();
 
-  /**
+  /** Returns The order to execute this coprocessor */
-   * @return The order to execute this coprocessor
-   */
   int getPriority();
 
-  /**
+  /** Returns Arbitrary key-value parameter pairs passed into the coprocessor. */
-   * @return Arbitrary key-value parameter pairs passed into the coprocessor.
-   */
   Map<String, String> getProperties();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

@@ -483,7 +483,7 @@ public class HBaseAdmin implements Admin {
     }
   }
 
-  /** @return Connection used by this object. */
+  /** Returns Connection used by this object. */
   @Override
   public Connection getConnection() {
     return connection;
@@ -3531,28 +3531,20 @@ public class HBaseAdmin implements Admin {
       return getDescription();
     }
 
-    /**
+    /** Returns the table name */
-     * @return the table name
-     */
     protected TableName getTableName() {
       return tableName;
     }
 
-    /**
+    /** Returns the table descriptor */
-     * @return the table descriptor
-     */
     protected TableDescriptor getTableDescriptor() throws IOException {
       return getAdmin().getDescriptor(getTableName());
     }
 
-    /**
+    /** Returns the operation type like CREATE, DELETE, DISABLE etc. */
-     * @return the operation type like CREATE, DELETE, DISABLE etc.
-     */
     public abstract String getOperationType();
 
-    /**
+    /** Returns a description of the operation */
-     * @return a description of the operation
-     */
     protected String getDescription() {
       return "Operation: " + getOperationType() + ", " + "Table Name: "
         + tableName.getNameWithNamespaceInclAsString() + ", procId: " + procId;
@@ -3696,16 +3688,12 @@ public class HBaseAdmin implements Admin {
       this.namespaceName = namespaceName;
     }
 
-    /**
+    /** Returns the namespace name */
-     * @return the namespace name
-     */
     protected String getNamespaceName() {
       return namespaceName;
     }
 
-    /**
+    /** Returns the operation type like CREATE_NAMESPACE, DELETE_NAMESPACE, etc. */
-     * @return the operation type like CREATE_NAMESPACE, DELETE_NAMESPACE, etc.
-     */
     public abstract String getOperationType();
 
     @Override

hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTable.java

@@ -204,9 +204,7 @@ public class HTable implements Table {
     this.locator = new HRegionLocator(tableName, connection);
   }
 
-  /**
+  /** Returns maxKeyValueSize from configuration. */
-   * @return maxKeyValueSize from configuration.
-   */
   public static int getMaxKeyValueSize(Configuration conf) {
     return conf.getInt(ConnectionConfiguration.MAX_KEYVALUE_SIZE_KEY, -1);
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTableMultiplexer.java

@@ -216,9 +216,7 @@ public class HTableMultiplexer {
     return put(TableName.valueOf(tableName), put);
   }
 
-  /**
+  /** Returns the current HTableMultiplexerStatus */
-   * @return the current HTableMultiplexerStatus
-   */
   public HTableMultiplexerStatus getHTableMultiplexerStatus() {
     return new HTableMultiplexerStatus(serverToFlushWorkerMap);
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Increment.java

@@ -162,9 +162,7 @@ public class Increment extends Mutation {
     return this;
   }
 
-  /**
+  /** Returns current setting for returnResults */
-   * @return current setting for returnResults
-   */
   // This method makes public the superclasses's protected method.
   @Override
   public boolean isReturnResults() {

hbase-client/src/main/java/org/apache/hadoop/hbase/client/MultiResponse.java

@@ -44,9 +44,7 @@ public class MultiResponse extends AbstractResponse {
     super();
   }
 
-  /**
+  /** Returns Number of pairs in this container */
-   * @return Number of pairs in this container
-   */
   public int size() {
     int size = 0;
     for (RegionResult result : results.values()) {
@@ -68,9 +66,7 @@ public class MultiResponse extends AbstractResponse {
     exceptions.put(regionName, ie);
   }
 
-  /**
+  /** Returns the exception for the region, if any. Null otherwise. */
-   * @return the exception for the region, if any. Null otherwise.
-   */
   public Throwable getException(byte[] regionName) {
     return exceptions.get(regionName);
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/MutableRegionInfo.java

@@ -127,14 +127,14 @@ class MutableRegionInfo implements RegionInfo {
   }
 
   /**
-   * @return Return a short, printable name for this region (usually encoded name) for us logging.
+   * Returns Return a short, printable name for this region (usually encoded name) for us logging.
    */
   @Override
   public String getShortNameToLog() {
     return RegionInfo.prettyPrint(this.getEncodedName());
   }
 
-  /** @return the regionId */
+  /** Returns the regionId */
   @Override
   public long getRegionId() {
     return regionId;
@@ -149,15 +149,13 @@ class MutableRegionInfo implements RegionInfo {
     return regionName;
   }
 
-  /**
+  /** Returns Region name as a String for use in logging, etc. */
-   * @return Region name as a String for use in logging, etc.
-   */
   @Override
   public String getRegionNameAsString() {
     return RegionInfo.getRegionNameAsString(this, this.regionName);
   }
 
-  /** @return the encoded region name */
+  /** Returns the encoded region name */
   @Override
   public String getEncodedName() {
     return this.encodedName;
@@ -168,13 +166,13 @@ class MutableRegionInfo implements RegionInfo {
     return this.encodedNameAsBytes;
   }
 
-  /** @return the startKey */
+  /** Returns the startKey */
   @Override
   public byte[] getStartKey() {
     return startKey;
   }
 
-  /** @return the endKey */
+  /** Returns the endKey */
   @Override
   public byte[] getEndKey() {
     return endKey;
@@ -219,15 +217,13 @@ class MutableRegionInfo implements RegionInfo {
         || Bytes.equals(endKey, HConstants.EMPTY_BYTE_ARRAY));
   }
 
-  /** @return true if this region is a meta region */
+  /** Returns true if this region is a meta region */
   @Override
   public boolean isMetaRegion() {
     return tableName.equals(TableName.META_TABLE_NAME);
   }
 
-  /**
+  /** Returns True if has been split and has daughters. */
-   * @return True if has been split and has daughters.
-   */
   @Override
   public boolean isSplit() {
     return this.split;

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Mutation.java

@@ -354,9 +354,7 @@ public abstract class Mutation extends OperationWithAttributes
     return this;
   }
 
-  /**
+  /** Returns the set of clusterIds that have consumed the mutation */
-   * @return the set of clusterIds that have consumed the mutation
-   */
   public List<UUID> getClusterIds() {
     List<UUID> clusterIds = new ArrayList<>();
     byte[] bytes = getAttribute(CONSUMED_CLUSTER_IDS);
@@ -379,9 +377,7 @@ public abstract class Mutation extends OperationWithAttributes
     return this;
   }
 
-  /**
+  /** Returns CellVisibility associated with cells in this Mutation. n */
-   * @return CellVisibility associated with cells in this Mutation. n
-   */
   public CellVisibility getCellVisibility() throws DeserializationException {
     byte[] cellVisibilityBytes = this.getAttribute(VisibilityConstants.VISIBILITY_LABELS_ATTR_KEY);
     if (cellVisibilityBytes == null) return null;
@@ -437,16 +433,12 @@ public abstract class Mutation extends OperationWithAttributes
     return size;
   }
 
-  /**
+  /** Returns the number of different families */
-   * @return the number of different families
-   */
   public int numFamilies() {
     return getFamilyCellMap().size();
   }
 
-  /**
+  /** Returns Calculate what Mutation adds to class heap size. */
-   * @return Calculate what Mutation adds to class heap size.
-   */
   @Override
   public long heapSize() {
     long heapsize = MUTATION_OVERHEAD;
@@ -475,9 +467,7 @@ public abstract class Mutation extends OperationWithAttributes
     return ClassSize.align(heapsize);
   }
 
-  /**
+  /** Returns The serialized ACL for this operation, or null if none */
-   * @return The serialized ACL for this operation, or null if none
-   */
   public byte[] getACL() {
     return getAttribute(AccessControlConstants.OP_ATTRIBUTE_ACL);
   }
@@ -527,9 +517,7 @@ public abstract class Mutation extends OperationWithAttributes
     return this;
   }
 
-  /**
+  /** Returns current value for returnResults */
-   * @return current value for returnResults
-   */
   // Used by Increment and Append only.
   @InterfaceAudience.Private
   protected boolean isReturnResults() {

hbase-client/src/main/java/org/apache/hadoop/hbase/client/NonceGenerator.java

@@ -29,9 +29,9 @@ public interface NonceGenerator {
 
   static final String CLIENT_NONCES_ENABLED_KEY = "hbase.client.nonces.enabled";
 
-  /** @return the nonce group (client ID) of this client manager. */
+  /** Returns the nonce group (client ID) of this client manager. */
   long getNonceGroup();
 
-  /** @return New nonce. */
+  /** Returns New nonce. */
   long newNonce();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Query.java

@@ -75,18 +75,14 @@ public abstract class Query extends OperationWithAttributes {
     return this;
   }
 
-  /**
+  /** Returns The authorizations this Query is associated with. n */
-   * @return The authorizations this Query is associated with. n
-   */
   public Authorizations getAuthorizations() throws DeserializationException {
     byte[] authorizationsBytes = this.getAttribute(VisibilityConstants.VISIBILITY_LABELS_ATTR_KEY);
     if (authorizationsBytes == null) return null;
     return ProtobufUtil.toAuthorizations(authorizationsBytes);
   }
 
-  /**
+  /** Returns The serialized ACL for this operation, or null if none */
-   * @return The serialized ACL for this operation, or null if none
-   */
   public byte[] getACL() {
     return getAttribute(AccessControlConstants.OP_ATTRIBUTE_ACL);
   }
@@ -220,9 +216,7 @@ public abstract class Query extends OperationWithAttributes {
     return this;
   }
 
-  /**
+  /** Returns A map of column families to time ranges */
-   * @return A map of column families to time ranges
-   */
   public Map<byte[], TimeRange> getColumnFamilyTimeRange() {
     return this.colFamTimeRangeMap;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionAdminServiceCallable.java

@@ -114,9 +114,7 @@ public abstract class RegionAdminServiceCallable<T> implements RetryingCallable<
     }
   }
 
-  /**
+  /** Returns {@link Connection} instance used by this Callable. */
-   * @return {@link Connection} instance used by this Callable.
-   */
   Connection getConnection() {
     return this.connection;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java

@@ -152,13 +152,11 @@ public interface RegionInfo extends Comparable<RegionInfo> {
   };
 
   /**
-   * @return Return a short, printable name for this region (usually encoded name) for us logging.
+   * Returns Return a short, printable name for this region (usually encoded name) for us logging.
    */
   String getShortNameToLog();
 
-  /**
+  /** Returns the regionId. */
-   * @return the regionId.
-   */
   long getRegionId();
 
   /**
@@ -167,44 +165,28 @@ public interface RegionInfo extends Comparable<RegionInfo> {
    */
   byte[] getRegionName();
 
-  /**
+  /** Returns Region name as a String for use in logging, etc. */
-   * @return Region name as a String for use in logging, etc.
-   */
   String getRegionNameAsString();
 
-  /**
+  /** Returns the encoded region name. */
-   * @return the encoded region name.
-   */
   String getEncodedName();
 
-  /**
+  /** Returns the encoded region name as an array of bytes. */
-   * @return the encoded region name as an array of bytes.
-   */
   byte[] getEncodedNameAsBytes();
 
-  /**
+  /** Returns the startKey. */
-   * @return the startKey.
-   */
   byte[] getStartKey();
 
-  /**
+  /** Returns the endKey. */
-   * @return the endKey.
-   */
   byte[] getEndKey();
 
-  /**
+  /** Returns current table name of the region */
-   * @return current table name of the region
-   */
   TableName getTable();
 
-  /**
+  /** Returns returns region replica id */
-   * @return returns region replica id
-   */
   int getReplicaId();
 
-  /**
+  /** Returns True if has been split and has daughters. */
-   * @return True if has been split and has daughters.
-   */
   boolean isSplit();
 
   /**
@@ -223,9 +205,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
   @Deprecated
   boolean isSplitParent();
 
-  /**
+  /** Returns true if this region is a meta region. */
-   * @return true if this region is a meta region.
-   */
   boolean isMetaRegion();
 
   /**
@@ -236,9 +216,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
    */
   boolean containsRange(byte[] rangeStartKey, byte[] rangeEndKey);
 
-  /**
+  /** Returns true if the given row falls in this region. */
-   * @return true if the given row falls in this region.
-   */
   boolean containsRow(byte[] row);
 
   /**
@@ -253,9 +231,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
       && (regionName[regionName.length - 1] == RegionInfo.ENC_SEPARATOR);
   }
 
-  /**
+  /** Returns the encodedName */
-   * @return the encodedName
-   */
   @InterfaceAudience.Private
   static String encodeRegionName(final byte[] regionName) {
     String encodedName;
@@ -371,7 +347,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
   }
 
   /**
-   * @return A deserialized {@link RegionInfo} or null if we failed deserialize or passed bytes null
+   * Returns A deserialized {@link RegionInfo} or null if we failed deserialize or passed bytes null
    */
   @InterfaceAudience.Private
   static RegionInfo parseFromOrNull(final byte[] bytes) {
@@ -380,7 +356,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
   }
 
   /**
-   * @return A deserialized {@link RegionInfo} or null if we failed deserialize or passed bytes null
+   * Returns A deserialized {@link RegionInfo} or null if we failed deserialize or passed bytes null
    */
   @InterfaceAudience.Private
   static RegionInfo parseFromOrNull(final byte[] bytes, int offset, int len) {
@@ -778,16 +754,12 @@ public interface RegionInfo extends Comparable<RegionInfo> {
     return ris;
   }
 
-  /**
+  /** Returns True if this is first Region in Table */
-   * @return True if this is first Region in Table
-   */
   default boolean isFirst() {
     return Bytes.equals(getStartKey(), HConstants.EMPTY_START_ROW);
   }
 
-  /**
+  /** Returns True if this is last Region in Table */
-   * @return True if this is last Region in Table
-   */
   default boolean isLast() {
     return Bytes.equals(getEndKey(), HConstants.EMPTY_END_ROW);
   }
@@ -809,9 +781,7 @@ public interface RegionInfo extends Comparable<RegionInfo> {
     return getTable().equals(other.getTable()) && areAdjacent(this, other);
   }
 
-  /**
+  /** Returns True if RegionInfo is degenerate... if startKey > endKey. */
-   * @return True if RegionInfo is degenerate... if startKey > endKey.
-   */
   default boolean isDegenerate() {
     return !isLast() && Bytes.compareTo(getStartKey(), getEndKey()) > 0;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionReplicaUtil.java

@@ -73,12 +73,12 @@ public class RegionReplicaUtil {
     return getRegionInfoForReplica(regionInfo, DEFAULT_REPLICA_ID);
   }
 
-  /** @return true if this replicaId corresponds to default replica for the region */
+  /** Returns true if this replicaId corresponds to default replica for the region */
   public static boolean isDefaultReplica(int replicaId) {
     return DEFAULT_REPLICA_ID == replicaId;
   }
 
-  /** @return true if this region is a default replica for the region */
+  /** Returns true if this region is a default replica for the region */
   public static boolean isDefaultReplica(RegionInfo hri) {
     return hri.getReplicaId() == DEFAULT_REPLICA_ID;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionServerCallable.java

@@ -155,9 +155,7 @@ public abstract class RegionServerCallable<T, S> implements RetryingCallable<T>
     }
   }
 
-  /**
+  /** Returns {@link ClusterConnection} instance used by this Callable. */
-   * @return {@link ClusterConnection} instance used by this Callable.
-   */
   protected ClusterConnection getConnection() {
     return (ClusterConnection) this.connection;
   }
@@ -200,9 +198,7 @@ public abstract class RegionServerCallable<T, S> implements RetryingCallable<T>
     return ConnectionUtils.getPauseTime(pause, tries);
   }
 
-  /**
+  /** Returns the HRegionInfo for the current region */
-   * @return the HRegionInfo for the current region
-   */
   public HRegionInfo getHRegionInfo() {
     if (this.location == null) {
       return null;

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RequestController.java

@@ -67,9 +67,7 @@ public interface RequestController {
     void reset() throws InterruptedIOException;
   }
 
-  /**
+  /** Returns A new checker for evaluating a batch rows. */
-   * @return A new checker for evaluating a batch rows.
-   */
   Checker newChecker();
 
   /**
@@ -86,9 +84,7 @@ public interface RequestController {
    */
   void decTaskCounters(Collection<byte[]> regions, ServerName sn);
 
-  /**
+  /** Returns The number of running task. */
-   * @return The number of running task.
-   */
   long getNumberOfTasksInProgress();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java

@@ -671,9 +671,7 @@ public class Result implements CellScannable, CellScanner {
     return this.cells == null || this.cells.length == 0;
   }
 
-  /**
+  /** Returns the size of the underlying Cell [] */
-   * @return the size of the underlying Cell []
-   */
   public int size() {
     return this.cells == null ? 0 : this.cells.length;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ResultScanner.java

@@ -114,8 +114,6 @@ public interface ResultScanner extends Closeable, Iterable<Result> {
    */
   boolean renewLease();
 
-  /**
+  /** Returns the scan metrics, or {@code null} if we do not enable metrics. */
-   * @return the scan metrics, or {@code null} if we do not enable metrics.
-   */
   ScanMetrics getScanMetrics();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Row.java

@@ -28,9 +28,7 @@ import org.apache.yetus.audience.InterfaceAudience;
 public interface Row extends Comparable<Row> {
   Comparator<Row> COMPARATOR = (v1, v2) -> Bytes.compareTo(v1.getRow(), v2.getRow());
 
-  /**
+  /** Returns The row. */
-   * @return The row.
-   */
   byte[] getRow();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RowAccess.java

@@ -26,13 +26,9 @@ import org.apache.yetus.audience.InterfaceAudience;
  */
 @InterfaceAudience.Private
 public interface RowAccess<T> extends Iterable<T> {
-  /**
+  /** Returns true if there are no elements. */
-   * @return true if there are no elements.
-   */
   boolean isEmpty();
 
-  /**
+  /** Returns the number of elements in this list. */
-   * @return the number of elements in this list.
-   */
   int size();
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RowMutations.java

@@ -155,9 +155,7 @@ public class RowMutations implements Row {
     return row;
   }
 
-  /**
+  /** Returns An unmodifiable list of the current mutations. */
-   * @return An unmodifiable list of the current mutations.
-   */
   public List<Mutation> getMutations() {
     return Collections.unmodifiableList(mutations);
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/RpcRetryingCallerImpl.java

@@ -176,9 +176,7 @@ public class RpcRetryingCallerImpl<T> implements RpcRetryingCaller<T> {
     }
   }
 
-  /**
+  /** Returns Calculate how long a single call took */
-   * @return Calculate how long a single call took
-   */
   private long singleCallDuration(final long expectedSleep) {
     return (EnvironmentEdgeManager.currentTime() - tracker.getStartTime()) + expectedSleep;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/Scan.java

@@ -687,9 +687,7 @@ public class Scan extends Query {
     return this;
   }
 
-  /**
+  /** Returns the maximum result size in bytes. See {@link #setMaxResultSize(long)} */
-   * @return the maximum result size in bytes. See {@link #setMaxResultSize(long)}
-   */
   public long getMaxResultSize() {
     return maxResultSize;
   }
@@ -727,9 +725,7 @@ public class Scan extends Query {
     return this.familyMap;
   }
 
-  /**
+  /** Returns the number of families in familyMap */
-   * @return the number of families in familyMap
-   */
   public int numFamilies() {
     if (hasFamilies()) {
       return this.familyMap.size();
@@ -737,16 +733,12 @@ public class Scan extends Query {
     return 0;
   }
 
-  /**
+  /** Returns true if familyMap is non empty, false otherwise */
-   * @return true if familyMap is non empty, false otherwise
-   */
   public boolean hasFamilies() {
     return !this.familyMap.isEmpty();
   }
 
-  /**
+  /** Returns the keys of the familyMap */
-   * @return the keys of the familyMap
-   */
   public byte[][] getFamilies() {
     if (hasFamilies()) {
       return this.familyMap.keySet().toArray(new byte[0][0]);
@@ -754,51 +746,37 @@ public class Scan extends Query {
     return null;
   }
 
-  /**
+  /** Returns the startrow */
-   * @return the startrow
-   */
   public byte[] getStartRow() {
     return this.startRow;
   }
 
-  /**
+  /** Returns if we should include start row when scan */
-   * @return if we should include start row when scan
-   */
   public boolean includeStartRow() {
     return includeStartRow;
   }
 
-  /**
+  /** Returns the stoprow */
-   * @return the stoprow
-   */
   public byte[] getStopRow() {
     return this.stopRow;
   }
 
-  /**
+  /** Returns if we should include stop row when scan */
-   * @return if we should include stop row when scan
-   */
   public boolean includeStopRow() {
     return includeStopRow;
   }
 
-  /**
+  /** Returns the max number of versions to fetch */
-   * @return the max number of versions to fetch
-   */
   public int getMaxVersions() {
     return this.maxVersions;
   }
 
-  /**
+  /** Returns maximum number of values to return for a single call to next() */
-   * @return maximum number of values to return for a single call to next()
-   */
   public int getBatch() {
     return this.batch;
   }
 
-  /**
+  /** Returns maximum number of values to return per row per CF */
-   * @return maximum number of values to return per row per CF
-   */
   public int getMaxResultsPerColumnFamily() {
     return this.storeLimit;
   }
@@ -811,9 +789,7 @@ public class Scan extends Query {
     return this.storeOffset;
   }
 
-  /**
+  /** Returns caching the number of rows fetched when calling next on a scanner */
-   * @return caching the number of rows fetched when calling next on a scanner
-   */
   public int getCaching() {
     return this.caching;
   }
@@ -833,9 +809,7 @@ public class Scan extends Query {
     return filter;
   }
 
-  /**
+  /** Returns true is a filter has been specified, false if not */
-   * @return true is a filter has been specified, false if not
-   */
   public boolean hasFilter() {
     return filter != null;
   }
@@ -996,9 +970,7 @@ public class Scan extends Query {
     return this;
   }
 
-  /**
+  /** Returns True if this Scan is in "raw" mode. */
-   * @return True if this Scan is in "raw" mode.
-   */
   public boolean isRaw() {
     byte[] attr = getAttribute(RAW_ATTR);
     return attr == null ? false : Bytes.toBoolean(attr);
@@ -1098,9 +1070,7 @@ public class Scan extends Query {
     return this;
   }
 
-  /**
+  /** Returns True if collection of scan metrics is enabled. For advanced users. */
-   * @return True if collection of scan metrics is enabled. For advanced users.
-   */
   public boolean isScanMetricsEnabled() {
     byte[] attr = getAttribute(Scan.SCAN_ATTRIBUTES_METRICS_ENABLE);
     return attr == null ? false : Bytes.toBoolean(attr);
@@ -1129,9 +1099,7 @@ public class Scan extends Query {
     return this;
   }
 
-  /**
+  /** Returns the limit of rows for this scan */
-   * @return the limit of rows for this scan
-   */
   public int getLimit() {
     return limit;
   }
@@ -1163,9 +1131,7 @@ public class Scan extends Query {
     PREAD
   }
 
-  /**
+  /** Returns the read type for this scan */
-   * @return the read type for this scan
-   */
   public ReadType getReadType() {
     return readType;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/ScannerCallable.java

@@ -396,9 +396,7 @@ public class ScannerCallable extends ClientServiceCallable<Result[]> {
     this.renew = val;
   }
 
-  /**
+  /** Returns the HRegionInfo for the current region */
-   * @return the HRegionInfo for the current region
-   */
   @Override
   public HRegionInfo getHRegionInfo() {
     if (!instantiated) {

hbase-client/src/main/java/org/apache/hadoop/hbase/client/TableDescriptor.java

@@ -153,9 +153,7 @@ public interface TableDescriptor {
   // those tables with the highest priority (From Yi Liang over on HBASE-18109).
   int getPriority();
 
-  /**
+  /** Returns Returns the configured replicas per region */
-   * @return Returns the configured replicas per region
-   */
   int getRegionReplication();
 
   /**
@@ -200,9 +198,7 @@ public interface TableDescriptor {
    */
   String getValue(String key);
 
-  /**
+  /** Returns Getter for fetching an unmodifiable map. */
-   * @return Getter for fetching an unmodifiable map.
-   */
   Map<Bytes, Bytes> getValues();
 
   /**
@@ -219,9 +215,7 @@ public interface TableDescriptor {
    */
   boolean hasColumnFamily(final byte[] name);
 
-  /**
+  /** Returns true if the read-replicas memstore replication is enabled. */
-   * @return true if the read-replicas memstore replication is enabled.
-   */
   boolean hasRegionMemStoreReplication();
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/client/TableDescriptorBuilder.java

@@ -1110,9 +1110,7 @@ public class TableDescriptorBuilder {
       return families.containsKey(familyName);
     }
 
-    /**
+    /** Returns Name of this table and then a map of all of the column family descriptors. */
-     * @return Name of this table and then a map of all of the column family descriptors.
-     */
     @Override
     public String toString() {
       StringBuilder s = new StringBuilder();
@@ -1135,9 +1133,7 @@ public class TableDescriptorBuilder {
       return s.toString();
     }
 
-    /**
+    /** Returns map of all table attributes formatted into string. */
-     * @return map of all table attributes formatted into string.
-     */
     public String toStringTableAttributes() {
       return getValues(true).toString();
     }
@@ -1247,9 +1243,7 @@ public class TableDescriptorBuilder {
       return false;
     }
 
-    /**
+    /** Returns hash code */
-     * @return hash code
-     */
     @Override
     public int hashCode() {
       int result = this.name.hashCode();
@@ -1297,9 +1291,7 @@ public class TableDescriptorBuilder {
       return setValue(REGION_REPLICATION_KEY, Integer.toString(regionReplication));
     }
 
-    /**
+    /** Returns true if the read-replicas memstore replication is enabled. */
-     * @return true if the read-replicas memstore replication is enabled.
-     */
     @Override
     public boolean hasRegionMemStoreReplication() {
       return getOrDefault(REGION_MEMSTORE_REPLICATION_KEY, Boolean::valueOf,
@@ -1550,9 +1542,7 @@ public class TableDescriptorBuilder {
       return getOrDefault(OWNER_KEY, Function.identity(), null);
     }
 
-    /**
+    /** Returns the bytes in pb format */
-     * @return the bytes in pb format
-     */
     private byte[] toByteArray() {
       return ProtobufUtil.prependPBMagic(ProtobufUtil.toTableSchema(this).toByteArray());
     }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/TableState.java

@@ -94,44 +94,32 @@ public class TableState {
   private final TableName tableName;
   private final State state;
 
-  /**
+  /** Returns True if table is {@link State#ENABLED}. */
-   * @return True if table is {@link State#ENABLED}.
-   */
   public boolean isEnabled() {
     return isInStates(State.ENABLED);
   }
 
-  /**
+  /** Returns True if table is {@link State#ENABLING}. */
-   * @return True if table is {@link State#ENABLING}.
-   */
   public boolean isEnabling() {
     return isInStates(State.ENABLING);
   }
 
-  /**
+  /** Returns True if {@link State#ENABLED} or {@link State#ENABLING} */
-   * @return True if {@link State#ENABLED} or {@link State#ENABLING}
-   */
   public boolean isEnabledOrEnabling() {
     return isInStates(State.ENABLED, State.ENABLING);
   }
 
-  /**
+  /** Returns True if table is disabled. */
-   * @return True if table is disabled.
-   */
   public boolean isDisabled() {
     return isInStates(State.DISABLED);
   }
 
-  /**
+  /** Returns True if table is disabling. */
-   * @return True if table is disabling.
-   */
   public boolean isDisabling() {
     return isInStates(State.DISABLING);
   }
 
-  /**
+  /** Returns True if {@link State#DISABLED} or {@link State#DISABLED} */
-   * @return True if {@link State#DISABLED} or {@link State#DISABLED}
-   */
   public boolean isDisabledOrDisabling() {
     return isInStates(State.DISABLED, State.DISABLING);
   }
@@ -146,9 +134,7 @@ public class TableState {
     this.state = state;
   }
 
-  /**
+  /** Returns table state */
-   * @return table state
-   */
   public State getState() {
     return state;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/client/backoff/ClientBackoffPolicy.java

@@ -32,8 +32,6 @@ public interface ClientBackoffPolicy {
 
   public static final String BACKOFF_POLICY_CLASS = "hbase.client.statistics.backoff-policy";
 
-  /**
+  /** Returns the number of ms to wait on the client based on the */
-   * @return the number of ms to wait on the client based on the
-   */
   public long getBackoffTime(ServerName serverName, byte[] region, ServerStatistics stats);
 }

hbase-client/src/main/java/org/apache/hadoop/hbase/exceptions/PreemptiveFastFailException.java

@@ -67,37 +67,27 @@ public class PreemptiveFastFailException extends ConnectException {
     this.guaranteedClientSideOnly = guaranteedClientSideOnly;
   }
 
-  /**
+  /** Returns time of the fist failure */
-   * @return time of the fist failure
-   */
   public long getFirstFailureAt() {
     return timeOfFirstFailureMilliSec;
   }
 
-  /**
+  /** Returns time of the latest attempt */
-   * @return time of the latest attempt
-   */
   public long getLastAttemptAt() {
     return timeOfLatestAttemptMilliSec;
   }
 
-  /**
+  /** Returns failure count */
-   * @return failure count
-   */
   public long getFailureCount() {
     return failureCount;
   }
 
-  /**
+  /** Returns true if operation was attempted by server, false otherwise. */
-   * @return true if operation was attempted by server, false otherwise.
-   */
   public boolean wasOperationAttemptedByServer() {
     return false;
   }
 
-  /**
+  /** Returns true if we know no mutation made it to the server, false otherwise. */
-   * @return true if we know no mutation made it to the server, false otherwise.
-   */
   public boolean isGuaranteedClientSideOnly() {
     return guaranteedClientSideOnly;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/BigDecimalComparator.java

@@ -72,9 +72,7 @@ public class BigDecimalComparator extends ByteArrayComparable {
     return this.bigDecimal.compareTo(that);
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.BigDecimalComparator.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/BinaryComparator.java

@@ -54,9 +54,7 @@ public class BinaryComparator extends org.apache.hadoop.hbase.filter.ByteArrayCo
     return ByteBufferUtils.compareTo(this.value, 0, this.value.length, value, offset, length);
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.BinaryComparator.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/BinaryComponentComparator.java

@@ -73,9 +73,7 @@ public class BinaryComponentComparator extends ByteArrayComparable {
     return result;
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.BinaryComponentComparator.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/BinaryPrefixComparator.java

@@ -58,9 +58,7 @@ public class BinaryPrefixComparator extends ByteArrayComparable {
     return ByteBufferUtils.compareTo(this.value, 0, this.value.length, value, offset, length);
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.BinaryPrefixComparator.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/BitComparator.java

@@ -57,16 +57,12 @@ public class BitComparator extends ByteArrayComparable {
     this.bitOperator = bitOperator;
   }
 
-  /**
+  /** Returns the bitwise operator */
-   * @return the bitwise operator
-   */
   public BitwiseOp getOperator() {
     return bitOperator;
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.BitComparator.Builder builder = ComparatorProtos.BitComparator.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnCountGetFilter.java

@@ -84,9 +84,7 @@ public class ColumnCountGetFilter extends FilterBase {
     return new ColumnCountGetFilter(limit);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.ColumnCountGetFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnPaginationFilter.java

@@ -157,9 +157,7 @@ public class ColumnPaginationFilter extends FilterBase {
     return new ColumnPaginationFilter(limit, offset);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.ColumnPaginationFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnPrefixFilter.java

@@ -108,9 +108,7 @@ public class ColumnPrefixFilter extends FilterBase {
     return new ColumnPrefixFilter(columnPrefix);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.ColumnPrefixFilter.Builder builder = FilterProtos.ColumnPrefixFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnRangeFilter.java

@@ -65,44 +65,32 @@ public class ColumnRangeFilter extends FilterBase {
     this.maxColumnInclusive = maxColumnInclusive;
   }
 
-  /**
+  /** Returns if min column range is inclusive. */
-   * @return if min column range is inclusive.
-   */
   public boolean isMinColumnInclusive() {
     return minColumnInclusive;
   }
 
-  /**
+  /** Returns if max column range is inclusive. */
-   * @return if max column range is inclusive.
-   */
   public boolean isMaxColumnInclusive() {
     return maxColumnInclusive;
   }
 
-  /**
+  /** Returns the min column range for the filter */
-   * @return the min column range for the filter
-   */
   public byte[] getMinColumn() {
     return this.minColumn;
   }
 
-  /**
+  /** Returns true if min column is inclusive, false otherwise */
-   * @return true if min column is inclusive, false otherwise
-   */
   public boolean getMinColumnInclusive() {
     return this.minColumnInclusive;
   }
 
-  /**
+  /** Returns the max column range for the filter */
-   * @return the max column range for the filter
-   */
   public byte[] getMaxColumn() {
     return this.maxColumn;
   }
 
-  /**
+  /** Returns true if max column is inclusive, false otherwise */
-   * @return true if max column is inclusive, false otherwise
-   */
   public boolean getMaxColumnInclusive() {
     return this.maxColumnInclusive;
   }
@@ -161,9 +149,7 @@ public class ColumnRangeFilter extends FilterBase {
     return new ColumnRangeFilter(minColumn, minColumnInclusive, maxColumn, maxColumnInclusive);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.ColumnRangeFilter.Builder builder = FilterProtos.ColumnRangeFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnValueFilter.java

@@ -75,23 +75,17 @@ public class ColumnValueFilter extends FilterBase {
     return op;
   }
 
-  /**
+  /** Returns the comparator */
-   * @return the comparator
-   */
   public ByteArrayComparable getComparator() {
     return comparator;
   }
 
-  /**
+  /** Returns the column family */
-   * @return the column family
-   */
   public byte[] getFamily() {
     return family;
   }
 
-  /**
+  /** Returns the qualifier */
-   * @return the qualifier
-   */
   public byte[] getQualifier() {
     return qualifier;
   }
@@ -161,9 +155,7 @@ public class ColumnValueFilter extends FilterBase {
     return new ColumnValueFilter(family, qualifier, operator, comparator);
   }
 
-  /**
+  /** Returns A pb instance to represent this instance. */
-   * @return A pb instance to represent this instance.
-   */
   FilterProtos.ColumnValueFilter convert() {
     FilterProtos.ColumnValueFilter.Builder builder = FilterProtos.ColumnValueFilter.newBuilder();
 

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/CompareFilter.java

@@ -113,9 +113,7 @@ public abstract class CompareFilter extends FilterBase {
     return op;
   }
 
-  /**
+  /** Returns the comparator */
-   * @return the comparator
-   */
   public ByteArrayComparable getComparator() {
     return comparator;
   }
@@ -279,9 +277,7 @@ public abstract class CompareFilter extends FilterBase {
     return arguments;
   }
 
-  /**
+  /** Returns A pb instance to represent this instance. */
-   * @return A pb instance to represent this instance.
-   */
   FilterProtos.CompareFilter convert() {
     FilterProtos.CompareFilter.Builder builder = FilterProtos.CompareFilter.newBuilder();
     HBaseProtos.CompareType compareOp = CompareType.valueOf(this.op.name());

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/DependentColumnFilter.java

@@ -112,23 +112,17 @@ public class DependentColumnFilter extends CompareFilter {
     this(family, qualifier, dropDependentColumn, CompareOp.NO_OP, null);
   }
 
-  /**
+  /** Returns the column family */
-   * @return the column family
-   */
   public byte[] getFamily() {
     return this.columnFamily;
   }
 
-  /**
+  /** Returns the column qualifier */
-   * @return the column qualifier
-   */
   public byte[] getQualifier() {
     return this.columnQualifier;
   }
 
-  /**
+  /** Returns true if we should drop the dependent column, false otherwise */
-   * @return true if we should drop the dependent column, false otherwise
-   */
   public boolean dropDependentColumn() {
     return this.dropDependentColumn;
   }
@@ -219,9 +213,7 @@ public class DependentColumnFilter extends CompareFilter {
     }
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.DependentColumnFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FamilyFilter.java

@@ -92,9 +92,7 @@ public class FamilyFilter extends CompareFilter {
     return new FamilyFilter(compareOp, comparator);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.FamilyFilter.Builder builder = FilterProtos.FamilyFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FilterList.java

@@ -192,9 +192,7 @@ final public class FilterList extends FilterBase {
     return filterListBase.filterRow();
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() throws IOException {
     FilterProtos.FilterList.Builder builder = FilterProtos.FilterList.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FirstKeyOnlyFilter.java

@@ -71,9 +71,7 @@ public class FirstKeyOnlyFilter extends FilterBase {
     return new FirstKeyOnlyFilter();
   }
 
-  /**
+  /** Returns true if first KV has been found. */
-   * @return true if first KV has been found.
-   */
   protected boolean hasFoundKV() {
     return this.foundKV;
   }
@@ -85,9 +83,7 @@ public class FirstKeyOnlyFilter extends FilterBase {
     this.foundKV = value;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.FirstKeyOnlyFilter.Builder builder = FilterProtos.FirstKeyOnlyFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FirstKeyValueMatchingQualifiersFilter.java

@@ -83,9 +83,7 @@ public class FirstKeyValueMatchingQualifiersFilter extends FirstKeyOnlyFilter {
     return false;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.FirstKeyValueMatchingQualifiersFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FuzzyRowFilter.java

@@ -284,9 +284,7 @@ public class FuzzyRowFilter extends FilterBase {
     return done;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.FuzzyRowFilter.Builder builder = FilterProtos.FuzzyRowFilter.newBuilder()

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/InclusiveStopFilter.java

@@ -81,9 +81,7 @@ public class InclusiveStopFilter extends FilterBase {
     return new InclusiveStopFilter(stopRowKey);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.InclusiveStopFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java

@@ -99,9 +99,7 @@ public class KeyOnlyFilter extends FilterBase {
     return filter;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.KeyOnlyFilter.Builder builder = FilterProtos.KeyOnlyFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/LongComparator.java

@@ -53,9 +53,7 @@ public class LongComparator extends ByteArrayComparable {
     return Long.compare(longValue, that);
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.LongComparator.Builder builder = ComparatorProtos.LongComparator.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/MultiRowRangeFilter.java

@@ -182,9 +182,7 @@ public class MultiRowRangeFilter extends FilterBase {
     return PrivateCellUtil.createFirstOnRow(comparisonData, 0, (short) comparisonData.length);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.MultiRowRangeFilter.Builder builder =
@@ -475,16 +473,12 @@ public class MultiRowRangeFilter extends FilterBase {
       return stopRow;
     }
 
-    /**
+    /** Returns if start row is inclusive. */
-     * @return if start row is inclusive.
-     */
     public boolean isStartRowInclusive() {
       return startRowInclusive;
     }
 
-    /**
+    /** Returns if stop row is inclusive. */
-     * @return if stop row is inclusive.
-     */
     public boolean isStopRowInclusive() {
       return stopRowInclusive;
     }

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/MultipleColumnPrefixFilter.java

@@ -122,9 +122,7 @@ public class MultipleColumnPrefixFilter extends FilterBase {
     return new MultipleColumnPrefixFilter(prefixes);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.MultipleColumnPrefixFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/NullComparator.java

@@ -63,9 +63,7 @@ public class NullComparator extends ByteArrayComparable {
     return value != null ? 1 : 0;
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.NullComparator.Builder builder = ComparatorProtos.NullComparator.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/PageFilter.java

@@ -97,9 +97,7 @@ public class PageFilter extends FilterBase {
     return new PageFilter(pageSize);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.PageFilter.Builder builder = FilterProtos.PageFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/PrefixFilter.java

@@ -107,9 +107,7 @@ public class PrefixFilter extends FilterBase {
     return new PrefixFilter(prefix);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.PrefixFilter.Builder builder = FilterProtos.PrefixFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/QualifierFilter.java

@@ -86,9 +86,7 @@ public class QualifierFilter extends CompareFilter {
     return new QualifierFilter(compareOp, comparator);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.QualifierFilter.Builder builder = FilterProtos.QualifierFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/RandomRowFilter.java

@@ -44,9 +44,7 @@ public class RandomRowFilter extends FilterBase {
     this.chance = chance;
   }
 
-  /**
+  /** Returns The chance that a row gets included. */
-   * @return The chance that a row gets included.
-   */
   public float getChance() {
     return chance;
   }
@@ -107,9 +105,7 @@ public class RandomRowFilter extends FilterBase {
     filterOutRow = false;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.RandomRowFilter.Builder builder = FilterProtos.RandomRowFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/RegexStringComparator.java

@@ -145,9 +145,7 @@ public class RegexStringComparator extends ByteArrayComparable {
     return engine.compareTo(value, offset, length);
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     return engine.toByteArray();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/RowFilter.java

@@ -106,9 +106,7 @@ public class RowFilter extends CompareFilter {
     return new RowFilter(compareOp, comparator);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.RowFilter.Builder builder = FilterProtos.RowFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/SingleColumnValueExcludeFilter.java

@@ -163,9 +163,7 @@ public class SingleColumnValueExcludeFilter extends SingleColumnValueFilter {
     return filter;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.SingleColumnValueExcludeFilter.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/SingleColumnValueFilter.java

@@ -192,23 +192,17 @@ public class SingleColumnValueFilter extends FilterBase {
     return op;
   }
 
-  /**
+  /** Returns the comparator */
-   * @return the comparator
-   */
   public org.apache.hadoop.hbase.filter.ByteArrayComparable getComparator() {
     return comparator;
   }
 
-  /**
+  /** Returns the family */
-   * @return the family
-   */
   public byte[] getFamily() {
     return columnFamily;
   }
 
-  /**
+  /** Returns the qualifier */
-   * @return the qualifier
-   */
   public byte[] getQualifier() {
     return columnQualifier;
   }
@@ -356,9 +350,7 @@ public class SingleColumnValueFilter extends FilterBase {
     return builder.build();
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     return convert().toByteArray();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/SkipFilter.java

@@ -102,9 +102,7 @@ public class SkipFilter extends FilterBase {
     return true;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() throws IOException {
     FilterProtos.SkipFilter.Builder builder = FilterProtos.SkipFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/SubstringComparator.java

@@ -66,9 +66,7 @@ public class SubstringComparator extends ByteArrayComparable {
     return Bytes.toString(value, offset, length).toLowerCase(Locale.ROOT).contains(substr) ? 0 : 1;
   }
 
-  /**
+  /** Returns The comparator serialized using pb */
-   * @return The comparator serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     ComparatorProtos.SubstringComparator.Builder builder =

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/TimestampsFilter.java

@@ -77,9 +77,7 @@ public class TimestampsFilter extends FilterBase {
     init();
   }
 
-  /**
+  /** Returns the list of timestamps */
-   * @return the list of timestamps
-   */
   public List<Long> getTimestamps() {
     List<Long> list = new ArrayList<>(timestamps.size());
     list.addAll(timestamps);
@@ -165,9 +163,7 @@ public class TimestampsFilter extends FilterBase {
     return new TimestampsFilter(timestamps);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.TimestampsFilter.Builder builder = FilterProtos.TimestampsFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ValueFilter.java

@@ -87,9 +87,7 @@ public class ValueFilter extends CompareFilter {
     return new ValueFilter(compareOp, comparator);
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() {
     FilterProtos.ValueFilter.Builder builder = FilterProtos.ValueFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/WhileMatchFilter.java

@@ -107,9 +107,7 @@ public class WhileMatchFilter extends FilterBase {
     return true;
   }
 
-  /**
+  /** Returns The filter serialized using pb */
-   * @return The filter serialized using pb
-   */
   @Override
   public byte[] toByteArray() throws IOException {
     FilterProtos.WhileMatchFilter.Builder builder = FilterProtos.WhileMatchFilter.newBuilder();

hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRpcController.java

@@ -60,9 +60,7 @@ public interface HBaseRpcController extends RpcController, CellScannable {
    */
   void setPriority(final TableName tn);
 
-  /**
+  /** Returns The priority of this request */
-   * @return The priority of this request
-   */
   int getPriority();
 
   int getCallTimeout();
@@ -110,16 +108,12 @@ public interface HBaseRpcController extends RpcController, CellScannable {
    */
   void notifyOnCancel(RpcCallback<Object> callback, CancellationCallback action) throws IOException;
 
-  /**
+  /** Returns True if this Controller is carrying the RPC target Region's RegionInfo. */
-   * @return True if this Controller is carrying the RPC target Region's RegionInfo.
-   */
   default boolean hasRegionInfo() {
     return false;
   }
 
-  /**
+  /** Returns Target Region's RegionInfo or null if not available or pertinent. */
-   * @return Target Region's RegionInfo or null if not available or pertinent.
-   */
   default RegionInfo getRegionInfo() {
     return null;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRpcControllerImpl.java

@@ -101,9 +101,7 @@ public class HBaseRpcControllerImpl implements HBaseRpcController {
     return this.regionInfo;
   }
 
-  /**
+  /** Returns One-shot cell scanner (you cannot back it up and restart) */
-   * @return One-shot cell scanner (you cannot back it up and restart)
-   */
   @Override
   public CellScanner cellScanner() {
     return cellScanner;

hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java

@@ -96,9 +96,7 @@ class IPCUtil {
     return totalSize;
   }
 
-  /**
+  /** Returns Size on the wire when the two messages are written with writeDelimitedTo */
-   * @return Size on the wire when the two messages are written with writeDelimitedTo
-   */
   public static int getTotalSizeWhenWrittenDelimited(Message... messages) {
     int totalSize = 0;
     for (Message m : messages) {
@@ -149,9 +147,7 @@ class IPCUtil {
         serverOverloaded);
   }
 
-  /**
+  /** Returns True if the exception is a fatal connection exception. */
-   * @return True if the exception is a fatal connection exception.
-   */
   static boolean isFatalConnectionException(final ExceptionResponse e) {
     return e.getExceptionClassName().equals(FatalConnectionException.class.getName());
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/RemoteWithExtrasException.java

@@ -123,30 +123,22 @@ public class RemoteWithExtrasException extends RemoteException {
     return ex;
   }
 
-  /**
+  /** Returns null if not set */
-   * @return null if not set
-   */
   public String getHostname() {
     return this.hostname;
   }
 
-  /**
+  /** Returns -1 if not set */
-   * @return -1 if not set
-   */
   public int getPort() {
     return this.port;
   }
 
-  /**
+  /** Returns True if origin exception was a do not retry type. */
-   * @return True if origin exception was a do not retry type.
-   */
   public boolean isDoNotRetry() {
     return this.doNotRetry;
   }
 
-  /**
+  /** Returns True if the server was considered overloaded when the exception was thrown. */
-   * @return True if the server was considered overloaded when the exception was thrown.
-   */
   public boolean isServerOverloaded() {
     return serverOverloaded;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufMagic.java

@@ -77,9 +77,7 @@ public class ProtobufMagic {
     return compareTo(PB_MAGIC, 0, PB_MAGIC.length, bytes, offset, PB_MAGIC.length) == 0;
   }
 
-  /**
+  /** Returns Length of {@link #PB_MAGIC} */
-   * @return Length of {@link #PB_MAGIC}
-   */
   public static int lengthOfPBMagic() {
     return PB_MAGIC.length;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java

@@ -223,9 +223,7 @@ public final class ProtobufUtil {
     }
   }
 
-  /**
+  /** Returns Length of {@link ProtobufMagic#lengthOfPBMagic()} */
-   * @return Length of {@link ProtobufMagic#lengthOfPBMagic()}
-   */
   public static int lengthOfPBMagic() {
     return ProtobufMagic.lengthOfPBMagic();
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/quotas/QuotaFilter.java

@@ -92,32 +92,32 @@ public class QuotaFilter {
     return this;
   }
 
-  /** @return true if the filter is empty */
+  /** Returns true if the filter is empty */
   public boolean isNull() {
     return !hasFilters;
   }
 
-  /** @return the QuotaType types that we want to filter one */
+  /** Returns the QuotaType types that we want to filter one */
   public Set<QuotaType> getTypeFilters() {
     return types;
   }
 
-  /** @return the Namespace filter regex */
+  /** Returns the Namespace filter regex */
   public String getNamespaceFilter() {
     return namespaceRegex;
   }
 
-  /** @return the Table filter regex */
+  /** Returns the Table filter regex */
   public String getTableFilter() {
     return tableRegex;
   }
 
-  /** @return the User filter regex */
+  /** Returns the User filter regex */
   public String getUserFilter() {
     return userRegex;
   }
 
-  /** @return the RegionServer filter regex */
+  /** Returns the RegionServer filter regex */
   public String getRegionServerFilter() {
     return regionServerRegex;
   }

hbase-client/src/main/java/org/apache/hadoop/hbase/quotas/SpaceQuotaSnapshot.java

@@ -74,9 +74,7 @@ public class SpaceQuotaSnapshot implements SpaceQuotaSnapshotView {
       return policy;
     }
 
-    /**
+    /** Returns {@code true} if the quota is being violated, {@code false} otherwise. */
-     * @return {@code true} if the quota is being violated, {@code false} otherwise.
-     */
     @Override
     public boolean isInViolation() {
       return inViolation;

hbase-client/src/main/java/org/apache/hadoop/hbase/quotas/SpaceQuotaSnapshotView.java

@@ -39,9 +39,7 @@ public interface SpaceQuotaSnapshotView {
      */
     Optional<SpaceViolationPolicy> getPolicy();
 
-    /**
+    /** Returns {@code true} if the quota is being violated, {@code false} otherwise. */
-     * @return {@code true} if the quota is being violated, {@code false} otherwise.
-     */
     boolean isInViolation();
   }
 

hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/Permission.java

@@ -203,7 +203,7 @@ public class Permission extends VersionedWritable {
     return raw.toString();
   }
 
-  /** @return the object version number */
+  /** Returns the object version number */
   @Override
   public byte getVersion() {
     return VERSION;