HBASE-22785 Fixed Checkstyle issues in exceptions and enhanced Javadoc

Signed-off-by: stack <stack@apache.org>

Author: HorizonNet

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

@@ -32,29 +32,29 @@ public class DoNotRetryIOException extends HBaseIOException {
   // TODO: This would be more useful as a marker interface than as a class.
   private static final long serialVersionUID = 1197446454511704139L;
 
-  /**
-   * default constructor
-   */
   public DoNotRetryIOException() {
     super();
   }
 
   /**
-   * @param message
+   * @param message the message for this exception
    */
   public DoNotRetryIOException(String message) {
     super(message);
   }
 
   /**
-   * @param message
-   * @param cause
+   * @param message the message for this exception
+   * @param throwable the {@link Throwable} to use for this exception
    */
-  public DoNotRetryIOException(String message, Throwable cause) {
-    super(message, cause);
+  public DoNotRetryIOException(String message, Throwable throwable) {
+    super(message, throwable);
   }
 
-  public DoNotRetryIOException(Throwable cause) {
-    super(cause);
+  /**
+   * @param throwable the {@link Throwable} to use for this exception
+   */
+  public DoNotRetryIOException(Throwable throwable) {
+    super(throwable);
   }
 }

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

@@ -23,28 +23,23 @@
 
 import java.io.IOException;
 
-
 /**
  * Thrown during flush if the possibility snapshot content was not properly
  * persisted into store files.  Response should include replay of wal content.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
 public class DroppedSnapshotException extends IOException {
-
   private static final long serialVersionUID = -5463156580831677374L;
 
-  /**
-   * @param msg
-   */
-  public DroppedSnapshotException(String msg) {
-    super(msg);
+  public DroppedSnapshotException() {
+    super();
   }
 
   /**
-   * default constructor
+   * @param message the message for this exception
    */
-  public DroppedSnapshotException() {
-    super();
+  public DroppedSnapshotException(String message) {
+    super(message);
   }
 }

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

@@ -18,39 +18,35 @@
  */
 package org.apache.hadoop.hbase;
 
+import java.io.IOException;
+
 import org.apache.hadoop.hbase.classification.InterfaceAudience;
 import org.apache.hadoop.hbase.classification.InterfaceStability;
 import org.apache.hadoop.hbase.util.Bytes;
 
-import java.io.IOException;
-
 /**
- * Thrown by a region server if it is sent a request for a region it is not
- * serving.
+ * Thrown by a region server if it is sent a request for a region it is not serving.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
 public class NotServingRegionException extends IOException {
   private static final long serialVersionUID = (1L << 17) - 1L;
 
-  /** default constructor */
   public NotServingRegionException() {
     super();
   }
 
   /**
-   * Constructor
-   * @param s message
+   * @param message the message for this exception
    */
-  public NotServingRegionException(String s) {
-    super(s);
+  public NotServingRegionException(String message) {
+    super(message);
   }
 
   /**
-   * Constructor
-   * @param s message
+   * @param message the message for this exception
    */
-  public NotServingRegionException(final byte [] s) {
-    super(Bytes.toString(s));
+  public NotServingRegionException(final byte[] message) {
+    super(Bytes.toString(message));
   }
 }

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

@@ -23,27 +23,28 @@
 import org.apache.hadoop.hbase.TableName;
 
 /**
- * Thrown when a table exists but should not
+ * Thrown when a table exists but should not.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
 public class TableExistsException extends DoNotRetryIOException {
   private static final long serialVersionUID = (1L << 7) - 1L;
-  /** default constructor */
+
   public TableExistsException() {
     super();
   }
 
   /**
-   * Constructor
-   *
-   * @param s message
+   * @param tableName the name of the table that should not exist
    */
-  public TableExistsException(String s) {
-    super(s);
+  public TableExistsException(String tableName) {
+    super(tableName);
   }
 
-  public TableExistsException(TableName t) {
-    this(t.getNameAsString());
+  /**
+   * @param tableName the name of the table that should not exist
+   */
+  public TableExistsException(TableName tableName) {
+    this(tableName.getNameAsString());
   }
 }

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

@@ -21,29 +21,38 @@
 import org.apache.hadoop.hbase.classification.InterfaceStability;
 
 /**
- * 
- * Failed to find .tableinfo file under table dir
- *
+ * Failed to find {@code .tableinfo} file under the table directory.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Evolving
 @SuppressWarnings("serial")
 public class TableInfoMissingException extends HBaseIOException {
-
+  /**
+   * Failed to find {@code .tableinfo} file under the table directory.
+   */
   public TableInfoMissingException() {
     super();
   }
 
-  public TableInfoMissingException( String message ) {
+  /**
+   * @param message the message for this exception
+   */
+  public TableInfoMissingException(String message) {
     super(message);
   }
 
-  public TableInfoMissingException( String message, Throwable t ) {
-    super(message, t);
+  /**
+   * @param message the message for this exception
+   * @param throwable the {@link Throwable} to use for this exception
+   */
+  public TableInfoMissingException(String message, Throwable throwable) {
+    super(message, throwable);
   }
 
-  public TableInfoMissingException( Throwable t ) {
-    super(t);
+  /**
+   * @param throwable the {@link Throwable} to use for this exception
+   */
+  public TableInfoMissingException(Throwable throwable) {
+    super(throwable);
   }
-
 }

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

@@ -18,40 +18,39 @@
  */
 package org.apache.hadoop.hbase;
 
+import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.classification.InterfaceAudience;
 import org.apache.hadoop.hbase.classification.InterfaceStability;
-import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.util.Bytes;
 
 /**
- * Thrown if a table should be offline but is not
+ * Thrown if a table should be offline but is not.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
 public class TableNotDisabledException extends DoNotRetryIOException {
   private static final long serialVersionUID = (1L << 19) - 1L;
-  /** default constructor */
+
   public TableNotDisabledException() {
     super();
   }
 
   /**
-   * Constructor
-   * @param s message
+   * @param tableName the name of the table that is not disabled
    */
-  public TableNotDisabledException(String s) {
-    super(s);
+  public TableNotDisabledException(String tableName) {
+    super(tableName);
   }
 
   /**
-   * @param tableName Name of table that is not disabled
+   * @param tableName the name of the table that is not disabled
    */
   public TableNotDisabledException(byte[] tableName) {
     this(Bytes.toString(tableName));
   }
 
   /**
-   * @param tableName Name of table that is not disabled
+   * @param tableName the name of the table that is not disabled
    */
   public TableNotDisabledException(TableName tableName) {
     this(tableName.getNameAsString());

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

@@ -18,14 +18,13 @@
  */
 package org.apache.hadoop.hbase;
 
+import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.classification.InterfaceAudience;
 import org.apache.hadoop.hbase.classification.InterfaceStability;
-import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.util.Bytes;
 
-
 /**
- * Thrown if a table should be enabled but is not
+ * Thrown if a table should be enabled but is not.
  */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
@@ -37,22 +36,21 @@ public TableNotEnabledException() {
   }
 
   /**
-   * Constructor
-   * @param s message
+   * @param tableName the name of table that is not enabled
    */
-  public TableNotEnabledException(String s) {
-    super(s);
+  public TableNotEnabledException(String tableName) {
+    super(tableName);
   }
 
   /**
-   * @param tableName Name of table that is not enabled
+   * @param tableName the name of table that is not enabled
    */
   public TableNotEnabledException(TableName tableName) {
     this(tableName.getNameAsString());
   }
 
   /**
-   * @param tableName Name of table that is not enabled
+   * @param tableName the name of table that is not enabled
    */
   public TableNotEnabledException(byte[] tableName) {
     this(Bytes.toString(tableName));

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

@@ -18,31 +18,40 @@
  */
 package org.apache.hadoop.hbase;
 
+import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.classification.InterfaceAudience;
 import org.apache.hadoop.hbase.classification.InterfaceStability;
-import org.apache.hadoop.hbase.TableName;
 import org.apache.hadoop.hbase.util.Bytes;
 
-/** Thrown when a table can not be located */
+/**
+ * Thrown when a table cannot be located.
+ */
 @InterfaceAudience.Public
 @InterfaceStability.Stable
 public class TableNotFoundException extends DoNotRetryIOException {
   private static final long serialVersionUID = 993179627856392526L;
 
-  /** default constructor */
   public TableNotFoundException() {
     super();
   }
 
-  /** @param s message */
-  public TableNotFoundException(String s) {
-    super(s);
+  /**
+   * @param tableName the name of the table which was not found
+   */
+  public TableNotFoundException(String tableName) {
+    super(tableName);
   }
 
+  /**
+   * @param tableName the name of the table which was not found
+   */
   public TableNotFoundException(byte[] tableName) {
     super(Bytes.toString(tableName));
   }
 
+  /**
+   * @param tableName the name of the table which was not found
+   */
   public TableNotFoundException(TableName tableName) {
     super(tableName.getNameAsString());
   }

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

@@ -30,6 +30,9 @@
 public class UnknownRegionException extends DoNotRetryRegionException {
   private static final long serialVersionUID = 1968858760475205392L;
 
+  /**
+   * @param regionName the name of the region which is unknown
+   */
   public UnknownRegionException(String regionName) {
     super(regionName);
   }