You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by al...@apache.org on 2018/12/28 07:33:40 UTC

[1/2] hbase git commit: HBASE-21650 Add DDL operation and some other miscellaneous to thrift2 (ADDENDUM add some comments)

Repository: hbase
Updated Branches:
  refs/heads/master 7820ba1db -> b620334c2


http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableDescriptor.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableDescriptor.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableDescriptor.java
index 89a8a5e..8e53bdf 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableDescriptor.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableDescriptor.java
@@ -34,7 +34,11 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.client.TableDescriptor
+ */
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TTableDescriptor implements org.apache.thrift.TBase<TTableDescriptor, TTableDescriptor._Fields>, java.io.Serializable, Cloneable, Comparable<TTableDescriptor> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TTableDescriptor");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableName.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableName.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableName.java
index f2c0743..cec268a 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableName.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTableName.java
@@ -34,7 +34,11 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.TableName
+ */
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TTableName implements org.apache.thrift.TBase<TTableName, TTableName._Fields>, java.io.Serializable, Cloneable, Comparable<TTableName> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TTableName");
 
@@ -47,12 +51,24 @@ public class TTableName implements org.apache.thrift.TBase<TTableName, TTableNam
     schemes.put(TupleScheme.class, new TTableNameTupleSchemeFactory());
   }
 
+  /**
+   * namespace name
+   */
   public ByteBuffer ns; // required
+  /**
+   * tablename
+   */
   public ByteBuffer qualifier; // required
 
   /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
   public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+    /**
+     * namespace name
+     */
     NS((short)1, "ns"),
+    /**
+     * tablename
+     */
     QUALIFIER((short)2, "qualifier");
 
     private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -157,6 +173,9 @@ public class TTableName implements org.apache.thrift.TBase<TTableName, TTableNam
     this.qualifier = null;
   }
 
+  /**
+   * namespace name
+   */
   public byte[] getNs() {
     setNs(org.apache.thrift.TBaseHelper.rightSize(ns));
     return ns == null ? null : ns.array();
@@ -166,6 +185,9 @@ public class TTableName implements org.apache.thrift.TBase<TTableName, TTableNam
     return org.apache.thrift.TBaseHelper.copyBinary(ns);
   }
 
+  /**
+   * namespace name
+   */
   public TTableName setNs(byte[] ns) {
     this.ns = ns == null ? (ByteBuffer)null : ByteBuffer.wrap(Arrays.copyOf(ns, ns.length));
     return this;
@@ -191,6 +213,9 @@ public class TTableName implements org.apache.thrift.TBase<TTableName, TTableNam
     }
   }
 
+  /**
+   * tablename
+   */
   public byte[] getQualifier() {
     setQualifier(org.apache.thrift.TBaseHelper.rightSize(qualifier));
     return qualifier == null ? null : qualifier.array();
@@ -200,6 +225,9 @@ public class TTableName implements org.apache.thrift.TBase<TTableName, TTableNam
     return org.apache.thrift.TBaseHelper.copyBinary(qualifier);
   }
 
+  /**
+   * tablename
+   */
   public TTableName setQualifier(byte[] qualifier) {
     this.qualifier = qualifier == null ? (ByteBuffer)null : ByteBuffer.wrap(Arrays.copyOf(qualifier, qualifier.length));
     return this;

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTimeRange.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTimeRange.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTimeRange.java
index 1e1898c..8ab746c 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTimeRange.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TTimeRange.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TTimeRange implements org.apache.thrift.TBase<TTimeRange, TTimeRange._Fields>, java.io.Serializable, Cloneable, Comparable<TTimeRange> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TTimeRange");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift b/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift
index 6383329..c1b94ef 100644
--- a/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift
+++ b/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift
@@ -315,6 +315,10 @@ enum TCompareOp {
   NO_OP = 6
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.regionserver.BloomType
+ */
 enum TBloomFilterType {
 /**
    * Bloomfilters disabled
@@ -338,6 +342,10 @@ enum TBloomFilterType {
   ROWPREFIX_DELIMITED = 4
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.io.compress.Algorithm
+ */
 enum TCompressionAlgorithm {
   LZO = 0,
   GZ = 1,
@@ -348,6 +356,10 @@ enum TCompressionAlgorithm {
   ZSTD = 6
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.io.encoding.DataBlockEncoding
+ */
 enum TDataBlockEncoding {
 /** Disable data block encoding. */
   NONE = 0,
@@ -361,6 +373,10 @@ enum TDataBlockEncoding {
   ROW_INDEX_V1 = 7
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.KeepDeletedCells
+ */
 enum TKeepDeletedCells {
   /** Deleted Cells are not retained. */
   FALSE = 0,
@@ -380,11 +396,21 @@ enum TKeepDeletedCells {
   TTL = 2
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.TableName
+ */
 struct TTableName {
+  /** namespace name */
   1: required binary ns
+  /** tablename */
   2: required binary qualifier
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.client.ColumnFamilyDescriptor
+ */
 struct TColumnFamilyDescriptor {
   1: required binary name
   2: optional map<binary, binary> attributes
@@ -409,6 +435,10 @@ struct TColumnFamilyDescriptor {
 
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.client.TableDescriptor
+ */
 struct TTableDescriptor {
  1: required TTableName tableName
  2: optional list<TColumnFamilyDescriptor> columns
@@ -416,6 +446,10 @@ struct TTableDescriptor {
  4: optional TDurability durability
 }
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.NamespaceDescriptor
+ */
 struct TNamespaceDescriptor {
 1: required string name
 2: optional map<string, string> configuration
@@ -761,110 +795,238 @@ service THBaseService {
     7: required TRowMutations rowMutations
   ) throws (1: TIOError io)
 
+  /**
+  * Get a table descriptor.
+  * @return the TableDescriptor of the giving tablename
+  **/
   TTableDescriptor getTableDescriptor(
-  1: required TTableName table
+    /** the tablename of the table to get tableDescriptor*/
+    1: required TTableName table
   ) throws (1: TIOError io)
 
+  /**
+  * Get table descriptors of tables.
+  * @return the TableDescriptor of the giving tablename
+  **/
   list<TTableDescriptor> getTableDescriptors(
+    /** the tablename list of the tables to get tableDescriptor*/
     1: required list<TTableName> tables
   ) throws (1: TIOError io)
 
+  /**
+  *
+  * @return true if table exists already, false if not
+  **/
   bool tableExists(
-  1: TTableName tableName
+    /** the tablename of the tables to check*/
+    1: TTableName tableName
   ) throws (1: TIOError io)
 
+  /**
+  * Get table descriptors of tables that match the given pattern
+  * @return the tableDescriptors of the matching table
+  **/
   list<TTableDescriptor> getTableDescriptorsByPattern(
+    /** The regular expression to match against */
     1: required string regex
+    /** set to false if match only against userspace tables */
     2: required bool includeSysTables
   ) throws (1: TIOError io)
 
+  /**
+  * Get table descriptors of tables in the given namespace
+  * @return the tableDescriptors in the namespce
+  **/
   list<TTableDescriptor> getTableDescriptorsByNamespace(
-      1: required string name
+    /** The namesapce's name */
+    1: required string name
   ) throws (1: TIOError io)
 
+  /**
+  * Get table names of tables that match the given pattern
+  * @return the table names of the matching table
+  **/
   list<TTableName> getTableNamesByPattern(
+    /** The regular expression to match against */
     1: required string regex
+    /** set to false if match only against userspace tables */
     2: required bool includeSysTables
   ) throws (1: TIOError io)
 
+  /**
+  * Get table names of tables in the given namespace
+  * @return the table names of the matching table
+  **/
   list<TTableName> getTableNamesByNamespace(
+    /** The namesapce's name */
     1: required string name
   ) throws (1: TIOError io)
 
+  /**
+  * Creates a new table with an initial set of empty regions defined by the specified split keys.
+  * The total number of regions created will be the number of split keys plus one. Synchronous
+  * operation.
+  **/
   void createTable(
+    /** table descriptor for table */
     1: required TTableDescriptor desc
+    /** rray of split keys for the initial regions of the table */
     2: required list<binary> splitKeys
   ) throws (1: TIOError io)
 
+  /**
+  * Deletes a table. Synchronous operation.
+  **/
   void deleteTable(
+    /** the tablename to delete */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+  /**
+  * Truncate a table. Synchronous operation.
+  **/
   void truncateTable(
+    /** the tablename to truncate */
     1: required TTableName tableName
+    /** whether to  preserve previous splits*/
     2: required bool preserveSplits
   ) throws (1: TIOError io)
 
+  /**
+  * Enalbe a table
+  **/
   void enableTable(
+    /** the tablename to enable */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+  /**
+  * Disable a table
+  **/
   void disableTable(
+    /** the tablename to disable */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+  /**
+  *
+  * @return true if table is enabled, false if not
+  **/
   bool isTableEnabled(
+    /** the tablename to check */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+ /**
+  *
+  * @return true if table is disabled, false if not
+  **/
   bool isTableDisabled(
+    /** the tablename to check */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+ /**
+  *
+  * @return true if table is available, false if not
+  **/
   bool isTableAvailable(
+    /** the tablename to check */
     1: required TTableName tableName
   ) throws (1: TIOError io)
 
+  /**
+   * Use this api to check if the table has been created with the specified number of splitkeys
+   * which was used while creating the given table. Note : If this api is used after a table's
+   * region gets splitted, the api may return false.
+   *
+   * @return true if table is available, false if not
+  **/
   bool isTableAvailableWithSplit(
+    /** the tablename to check */
     1: required TTableName tableName
+    /** keys to check if the table has been created with all split keys */
     2: required list<binary> splitKeys
   ) throws (1: TIOError io)
 
+  /**
+  * Add a column family to an existing table. Synchronous operation.
+  **/
   void addColumnFamily(
+    /** the tablename to add column family to */
     1: required TTableName tableName
+    /** column family descriptor of column family to be added */
     2: required TColumnFamilyDescriptor column
   ) throws (1: TIOError io)
 
+  /**
+  * Delete a column family from a table. Synchronous operation.
+  **/
   void deleteColumnFamily(
+    /** the tablename to delete column family from */
     1: required TTableName tableName
+    /** name of column family to be deleted */
     2: required binary column
   ) throws (1: TIOError io)
 
+  /**
+  * Modify an existing column family on a table. Synchronous operation.
+  **/
   void modifyColumnFamily(
+     /** the tablename to modify column family */
     1: required TTableName tableName
+    /** column family descriptor of column family to be modified */
     2: required TColumnFamilyDescriptor column
   ) throws (1: TIOError io)
 
+  /**
+  * Modify an existing table
+  **/
   void modifyTable(
+    /** the descriptor of the table to modify */
     1: required TTableDescriptor desc
   ) throws (1: TIOError io)
 
+  /**
+  * Create a new namespace. Blocks until namespace has been successfully created or an exception is
+  * thrown
+  **/
   void createNamespace(
+    /** descriptor which describes the new namespace */
     1: required TNamespaceDescriptor namespaceDesc
   ) throws (1: TIOError io)
 
+  /**
+  * Modify an existing namespace.  Blocks until namespace has been successfully modified or an
+  * exception is thrown
+  **/
   void modifyNamespace(
+    /** descriptor which describes the new namespace */
     1: required TNamespaceDescriptor namespaceDesc
   ) throws (1: TIOError io)
 
+  /**
+  * Delete an existing namespace. Only empty namespaces (no tables) can be removed.
+  * Blocks until namespace has been successfully deleted or an
+  * exception is thrown.
+  **/
   void deleteNamespace(
+    /** namespace name */
     1: required string name
   ) throws (1: TIOError io)
 
+  /**
+  *  Get a namespace descriptor by name.
+  *  @retrun the descriptor
+  **/
   TNamespaceDescriptor getNamespaceDescriptor(
+    /** name of namespace descriptor */
     1: required string name
   ) throws (1: TIOError io)
 
+  /**
+  * @return all namespaces
+  **/
   list<TNamespaceDescriptor> listNamespaceDescriptors(
   ) throws (1: TIOError io)
 }


[2/2] hbase git commit: HBASE-21650 Add DDL operation and some other miscellaneous to thrift2 (ADDENDUM add some comments)

Posted by al...@apache.org.
HBASE-21650 Add DDL operation and some other miscellaneous to thrift2 (ADDENDUM add some comments)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/b620334c
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/b620334c
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/b620334c

Branch: refs/heads/master
Commit: b620334c20e84a4876226b508213ce11b8b187a0
Parents: 7820ba1
Author: Allan Yang <al...@apache.org>
Authored: Fri Dec 28 15:32:50 2018 +0800
Committer: Allan Yang <al...@apache.org>
Committed: Fri Dec 28 15:32:50 2018 +0800

----------------------------------------------------------------------
 .../hbase/thrift/generated/AlreadyExists.java   |   2 +-
 .../hbase/thrift/generated/BatchMutation.java   |   2 +-
 .../thrift/generated/ColumnDescriptor.java      |   2 +-
 .../hadoop/hbase/thrift/generated/Hbase.java    |   2 +-
 .../hadoop/hbase/thrift/generated/IOError.java  |   2 +-
 .../hbase/thrift/generated/IllegalArgument.java |   2 +-
 .../hadoop/hbase/thrift/generated/Mutation.java |   2 +-
 .../hadoop/hbase/thrift/generated/TAppend.java  |   2 +-
 .../hadoop/hbase/thrift/generated/TCell.java    |   2 +-
 .../hadoop/hbase/thrift/generated/TColumn.java  |   2 +-
 .../hbase/thrift/generated/TIncrement.java      |   2 +-
 .../hbase/thrift/generated/TRegionInfo.java     |   2 +-
 .../hbase/thrift/generated/TRowResult.java      |   2 +-
 .../hadoop/hbase/thrift/generated/TScan.java    |   2 +-
 .../hadoop/hbase/thrift2/generated/TAppend.java |   2 +-
 .../hbase/thrift2/generated/TAuthorization.java |   2 +-
 .../thrift2/generated/TBloomFilterType.java     |   4 +
 .../thrift2/generated/TCellVisibility.java      |   2 +-
 .../hadoop/hbase/thrift2/generated/TColumn.java |   2 +-
 .../generated/TColumnFamilyDescriptor.java      |   6 +-
 .../thrift2/generated/TColumnIncrement.java     |   2 +-
 .../hbase/thrift2/generated/TColumnValue.java   |   2 +-
 .../generated/TCompressionAlgorithm.java        |   4 +
 .../thrift2/generated/TDataBlockEncoding.java   |   4 +
 .../hadoop/hbase/thrift2/generated/TDelete.java |   2 +-
 .../hadoop/hbase/thrift2/generated/TGet.java    |   2 +-
 .../hbase/thrift2/generated/THBaseService.java  | 571 ++++++++++++++++++-
 .../hbase/thrift2/generated/THRegionInfo.java   |   2 +-
 .../thrift2/generated/THRegionLocation.java     |   2 +-
 .../hbase/thrift2/generated/TIOError.java       |   2 +-
 .../thrift2/generated/TIllegalArgument.java     |   2 +-
 .../hbase/thrift2/generated/TIncrement.java     |   2 +-
 .../thrift2/generated/TKeepDeletedCells.java    |   4 +
 .../thrift2/generated/TNamespaceDescriptor.java |   6 +-
 .../hadoop/hbase/thrift2/generated/TPut.java    |   2 +-
 .../hadoop/hbase/thrift2/generated/TResult.java |   2 +-
 .../hbase/thrift2/generated/TRowMutations.java  |   2 +-
 .../hadoop/hbase/thrift2/generated/TScan.java   |   2 +-
 .../hbase/thrift2/generated/TServerName.java    |   2 +-
 .../thrift2/generated/TTableDescriptor.java     |   6 +-
 .../hbase/thrift2/generated/TTableName.java     |  30 +-
 .../hbase/thrift2/generated/TTimeRange.java     |   2 +-
 .../apache/hadoop/hbase/thrift2/hbase.thrift    | 168 +++++-
 43 files changed, 828 insertions(+), 41 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/AlreadyExists.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/AlreadyExists.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/AlreadyExists.java
index 8ec3e32..4457b9f 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/AlreadyExists.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/AlreadyExists.java
@@ -38,7 +38,7 @@ import org.slf4j.LoggerFactory;
  * An AlreadyExists exceptions signals that a table with the specified
  * name already exists
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class AlreadyExists extends TException implements org.apache.thrift.TBase<AlreadyExists, AlreadyExists._Fields>, java.io.Serializable, Cloneable, Comparable<AlreadyExists> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("AlreadyExists");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/BatchMutation.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/BatchMutation.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/BatchMutation.java
index 0872223..f605286 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/BatchMutation.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/BatchMutation.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * A BatchMutation object is used to apply a number of Mutations to a single row.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class BatchMutation implements org.apache.thrift.TBase<BatchMutation, BatchMutation._Fields>, java.io.Serializable, Cloneable, Comparable<BatchMutation> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("BatchMutation");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/ColumnDescriptor.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/ColumnDescriptor.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/ColumnDescriptor.java
index bccd48b..c0e885d 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/ColumnDescriptor.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/ColumnDescriptor.java
@@ -39,7 +39,7 @@ import org.slf4j.LoggerFactory;
  * such as the number of versions, compression settings, etc. It is
  * used as input when creating a table or adding a column.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class ColumnDescriptor implements org.apache.thrift.TBase<ColumnDescriptor, ColumnDescriptor._Fields>, java.io.Serializable, Cloneable, Comparable<ColumnDescriptor> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("ColumnDescriptor");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java
index dc4d8bc..54bb4ce 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class Hbase {
 
   public interface Iface {

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java
index a32b008..09d076c 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java
@@ -39,7 +39,7 @@ import org.slf4j.LoggerFactory;
  * to the Hbase master or an Hbase region server.  Also used to return
  * more general Hbase error conditions.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class IOError extends TException implements org.apache.thrift.TBase<IOError, IOError._Fields>, java.io.Serializable, Cloneable, Comparable<IOError> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("IOError");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IllegalArgument.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IllegalArgument.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IllegalArgument.java
index 573c496..bfe4380 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IllegalArgument.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/IllegalArgument.java
@@ -38,7 +38,7 @@ import org.slf4j.LoggerFactory;
  * An IllegalArgument exception indicates an illegal or invalid
  * argument was passed into a procedure.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class IllegalArgument extends TException implements org.apache.thrift.TBase<IllegalArgument, IllegalArgument._Fields>, java.io.Serializable, Cloneable, Comparable<IllegalArgument> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("IllegalArgument");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Mutation.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Mutation.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Mutation.java
index 72925b7..967e3d1 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Mutation.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/Mutation.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * A Mutation object is used to either update or delete a column-value.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class Mutation implements org.apache.thrift.TBase<Mutation, Mutation._Fields>, java.io.Serializable, Cloneable, Comparable<Mutation> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("Mutation");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TAppend.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TAppend.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TAppend.java
index a21b5f7..599f645 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TAppend.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TAppend.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * An Append object is used to specify the parameters for performing the append operation.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TAppend implements org.apache.thrift.TBase<TAppend, TAppend._Fields>, java.io.Serializable, Cloneable, Comparable<TAppend> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TAppend");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TCell.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TCell.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TCell.java
index afe0af1..c8f3576 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TCell.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TCell.java
@@ -40,7 +40,7 @@ import org.slf4j.LoggerFactory;
  * the timestamp of a cell to a first-class value, making it easy to take
  * note of temporal data. Cell is used all the way from HStore up to HTable.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TCell implements org.apache.thrift.TBase<TCell, TCell._Fields>, java.io.Serializable, Cloneable, Comparable<TCell> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TCell");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TColumn.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TColumn.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TColumn.java
index 0115bbd..0ce1d75 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TColumn.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TColumn.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Holds column name and the cell.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TColumn implements org.apache.thrift.TBase<TColumn, TColumn._Fields>, java.io.Serializable, Cloneable, Comparable<TColumn> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TColumn");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TIncrement.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TIncrement.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TIncrement.java
index 7c018f1..6c0827b 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TIncrement.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TIncrement.java
@@ -38,7 +38,7 @@ import org.slf4j.LoggerFactory;
  * For increments that are not incrementColumnValue
  * equivalents.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TIncrement implements org.apache.thrift.TBase<TIncrement, TIncrement._Fields>, java.io.Serializable, Cloneable, Comparable<TIncrement> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TIncrement");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRegionInfo.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRegionInfo.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRegionInfo.java
index ec75247..d2d3e0e 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRegionInfo.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRegionInfo.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * A TRegionInfo contains information about an HTable region.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TRegionInfo implements org.apache.thrift.TBase<TRegionInfo, TRegionInfo._Fields>, java.io.Serializable, Cloneable, Comparable<TRegionInfo> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TRegionInfo");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRowResult.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRowResult.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRowResult.java
index b09a368..6520007 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRowResult.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TRowResult.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Holds row name and then a map of columns to cells.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TRowResult implements org.apache.thrift.TBase<TRowResult, TRowResult._Fields>, java.io.Serializable, Cloneable, Comparable<TRowResult> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TRowResult");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java
index 97d59c7..fcc237d 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * A Scan object is used to specify scanner parameters when opening a scanner.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TScan implements org.apache.thrift.TBase<TScan, TScan._Fields>, java.io.Serializable, Cloneable, Comparable<TScan> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TScan");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAppend.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAppend.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAppend.java
index c89f67b..1b9d6fc 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAppend.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAppend.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TAppend implements org.apache.thrift.TBase<TAppend, TAppend._Fields>, java.io.Serializable, Cloneable, Comparable<TAppend> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TAppend");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAuthorization.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAuthorization.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAuthorization.java
index 1b168d8..916dc0c 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAuthorization.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TAuthorization.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TAuthorization implements org.apache.thrift.TBase<TAuthorization, TAuthorization._Fields>, java.io.Serializable, Cloneable, Comparable<TAuthorization> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TAuthorization");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TBloomFilterType.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TBloomFilterType.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TBloomFilterType.java
index 601d6b4..c4022a0 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TBloomFilterType.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TBloomFilterType.java
@@ -11,6 +11,10 @@ import java.util.Map;
 import java.util.HashMap;
 import org.apache.thrift.TEnum;
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.regionserver.BloomType
+ */
 public enum TBloomFilterType implements org.apache.thrift.TEnum {
   /**
    * Bloomfilters disabled

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCellVisibility.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCellVisibility.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCellVisibility.java
index 464ac12..e9af9fe 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCellVisibility.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCellVisibility.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TCellVisibility implements org.apache.thrift.TBase<TCellVisibility, TCellVisibility._Fields>, java.io.Serializable, Cloneable, Comparable<TCellVisibility> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TCellVisibility");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumn.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumn.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumn.java
index 24a7846..11e3ace 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumn.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumn.java
@@ -39,7 +39,7 @@ import org.slf4j.LoggerFactory;
  * in a HBase table by column family and optionally
  * a column qualifier and timestamp
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TColumn implements org.apache.thrift.TBase<TColumn, TColumn._Fields>, java.io.Serializable, Cloneable, Comparable<TColumn> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TColumn");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnFamilyDescriptor.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnFamilyDescriptor.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnFamilyDescriptor.java
index 03cb2f6..df4656f 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnFamilyDescriptor.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnFamilyDescriptor.java
@@ -34,7 +34,11 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.client.ColumnFamilyDescriptor
+ */
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TColumnFamilyDescriptor implements org.apache.thrift.TBase<TColumnFamilyDescriptor, TColumnFamilyDescriptor._Fields>, java.io.Serializable, Cloneable, Comparable<TColumnFamilyDescriptor> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TColumnFamilyDescriptor");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnIncrement.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnIncrement.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnIncrement.java
index 0f27519..18c5ff2 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnIncrement.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnIncrement.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Represents a single cell and the amount to increment it by
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TColumnIncrement implements org.apache.thrift.TBase<TColumnIncrement, TColumnIncrement._Fields>, java.io.Serializable, Cloneable, Comparable<TColumnIncrement> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TColumnIncrement");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnValue.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnValue.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnValue.java
index 6cded1b..1355141 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnValue.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TColumnValue.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * Represents a single cell and its value.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TColumnValue implements org.apache.thrift.TBase<TColumnValue, TColumnValue._Fields>, java.io.Serializable, Cloneable, Comparable<TColumnValue> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TColumnValue");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCompressionAlgorithm.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCompressionAlgorithm.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCompressionAlgorithm.java
index 46799be..59f7448 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCompressionAlgorithm.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TCompressionAlgorithm.java
@@ -11,6 +11,10 @@ import java.util.Map;
 import java.util.HashMap;
 import org.apache.thrift.TEnum;
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.io.compress.Algorithm
+ */
 public enum TCompressionAlgorithm implements org.apache.thrift.TEnum {
   LZO(0),
   GZ(1),

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDataBlockEncoding.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDataBlockEncoding.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDataBlockEncoding.java
index 1594aac3..bd513e9 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDataBlockEncoding.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDataBlockEncoding.java
@@ -11,6 +11,10 @@ import java.util.Map;
 import java.util.HashMap;
 import org.apache.thrift.TEnum;
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.io.encoding.DataBlockEncoding
+ */
 public enum TDataBlockEncoding implements org.apache.thrift.TEnum {
   /**
    * Disable data block encoding.

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDelete.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDelete.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDelete.java
index fb4ffe6..cdce4fb 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDelete.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDelete.java
@@ -60,7 +60,7 @@ import org.slf4j.LoggerFactory;
  * by changing the durability. If you don't provide durability, it defaults to
  * column family's default setting for durability.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TDelete implements org.apache.thrift.TBase<TDelete, TDelete._Fields>, java.io.Serializable, Cloneable, Comparable<TDelete> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TDelete");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TGet.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TGet.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TGet.java
index 1b01ef5..b38d936 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TGet.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TGet.java
@@ -47,7 +47,7 @@ import org.slf4j.LoggerFactory;
  * If you specify a time range and a timestamp the range is ignored.
  * Timestamps on TColumns are ignored.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TGet implements org.apache.thrift.TBase<TGet, TGet._Fields>, java.io.Serializable, Cloneable, Comparable<TGet> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TGet");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THBaseService.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THBaseService.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THBaseService.java
index 7388443..bb5db4d 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THBaseService.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THBaseService.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class THBaseService {
 
   public interface Iface {
@@ -282,54 +282,239 @@ public class THBaseService {
      */
     public boolean checkAndMutate(ByteBuffer table, ByteBuffer row, ByteBuffer family, ByteBuffer qualifier, TCompareOp compareOp, ByteBuffer value, TRowMutations rowMutations) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get a table descriptor.
+     * @return the TableDescriptor of the giving tablename
+     * 
+     * 
+     * @param table the tablename of the table to get tableDescriptor
+     */
     public TTableDescriptor getTableDescriptor(TTableName table) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get table descriptors of tables.
+     * @return the TableDescriptor of the giving tablename
+     * 
+     * 
+     * @param tables the tablename list of the tables to get tableDescriptor
+     */
     public List<TTableDescriptor> getTableDescriptors(List<TTableName> tables) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * 
+     * @return true if table exists already, false if not
+     * 
+     * 
+     * @param tableName the tablename of the tables to check
+     */
     public boolean tableExists(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get table descriptors of tables that match the given pattern
+     * @return the tableDescriptors of the matching table
+     * 
+     * 
+     * @param regex The regular expression to match against
+     * 
+     * @param includeSysTables set to false if match only against userspace tables
+     */
     public List<TTableDescriptor> getTableDescriptorsByPattern(String regex, boolean includeSysTables) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get table descriptors of tables in the given namespace
+     * @return the tableDescriptors in the namespce
+     * 
+     * 
+     * @param name The namesapce's name
+     */
     public List<TTableDescriptor> getTableDescriptorsByNamespace(String name) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get table names of tables that match the given pattern
+     * @return the table names of the matching table
+     * 
+     * 
+     * @param regex The regular expression to match against
+     * 
+     * @param includeSysTables set to false if match only against userspace tables
+     */
     public List<TTableName> getTableNamesByPattern(String regex, boolean includeSysTables) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get table names of tables in the given namespace
+     * @return the table names of the matching table
+     * 
+     * 
+     * @param name The namesapce's name
+     */
     public List<TTableName> getTableNamesByNamespace(String name) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Creates a new table with an initial set of empty regions defined by the specified split keys.
+     * The total number of regions created will be the number of split keys plus one. Synchronous
+     * operation.
+     * 
+     * 
+     * @param desc table descriptor for table
+     * 
+     * @param splitKeys rray of split keys for the initial regions of the table
+     */
     public void createTable(TTableDescriptor desc, List<ByteBuffer> splitKeys) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Deletes a table. Synchronous operation.
+     * 
+     * 
+     * @param tableName the tablename to delete
+     */
     public void deleteTable(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Truncate a table. Synchronous operation.
+     * 
+     * 
+     * @param tableName the tablename to truncate
+     * 
+     * @param preserveSplits whether to  preserve previous splits
+     */
     public void truncateTable(TTableName tableName, boolean preserveSplits) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Enalbe a table
+     * 
+     * 
+     * @param tableName the tablename to enable
+     */
     public void enableTable(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Disable a table
+     * 
+     * 
+     * @param tableName the tablename to disable
+     */
     public void disableTable(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * 
+     * @return true if table is enabled, false if not
+     * 
+     * 
+     * @param tableName the tablename to check
+     */
     public boolean isTableEnabled(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * 
+     * @return true if table is disabled, false if not
+     * 
+     * 
+     * @param tableName the tablename to check
+     */
     public boolean isTableDisabled(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * 
+     * @return true if table is available, false if not
+     * 
+     * 
+     * @param tableName the tablename to check
+     */
     public boolean isTableAvailable(TTableName tableName) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     *  * Use this api to check if the table has been created with the specified number of splitkeys
+     *  * which was used while creating the given table. Note : If this api is used after a table's
+     *  * region gets splitted, the api may return false.
+     *  *
+     *  * @return true if table is available, false if not
+     * *
+     * 
+     * @param tableName the tablename to check
+     * 
+     * @param splitKeys keys to check if the table has been created with all split keys
+     */
     public boolean isTableAvailableWithSplit(TTableName tableName, List<ByteBuffer> splitKeys) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Add a column family to an existing table. Synchronous operation.
+     * 
+     * 
+     * @param tableName the tablename to add column family to
+     * 
+     * @param column column family descriptor of column family to be added
+     */
     public void addColumnFamily(TTableName tableName, TColumnFamilyDescriptor column) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Delete a column family from a table. Synchronous operation.
+     * 
+     * 
+     * @param tableName the tablename to delete column family from
+     * 
+     * @param column name of column family to be deleted
+     */
     public void deleteColumnFamily(TTableName tableName, ByteBuffer column) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Modify an existing column family on a table. Synchronous operation.
+     * 
+     * 
+     * @param tableName the tablename to modify column family
+     * 
+     * @param column column family descriptor of column family to be modified
+     */
     public void modifyColumnFamily(TTableName tableName, TColumnFamilyDescriptor column) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Modify an existing table
+     * 
+     * 
+     * @param desc the descriptor of the table to modify
+     */
     public void modifyTable(TTableDescriptor desc) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Create a new namespace. Blocks until namespace has been successfully created or an exception is
+     * thrown
+     * 
+     * 
+     * @param namespaceDesc descriptor which describes the new namespace
+     */
     public void createNamespace(TNamespaceDescriptor namespaceDesc) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Modify an existing namespace.  Blocks until namespace has been successfully modified or an
+     * exception is thrown
+     * 
+     * 
+     * @param namespaceDesc descriptor which describes the new namespace
+     */
     public void modifyNamespace(TNamespaceDescriptor namespaceDesc) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Delete an existing namespace. Only empty namespaces (no tables) can be removed.
+     * Blocks until namespace has been successfully deleted or an
+     * exception is thrown.
+     * 
+     * 
+     * @param name namespace name
+     */
     public void deleteNamespace(String name) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * Get a namespace descriptor by name.
+     * @retrun the descriptor
+     * 
+     * 
+     * @param name name of namespace descriptor
+     */
     public TNamespaceDescriptor getNamespaceDescriptor(String name) throws TIOError, org.apache.thrift.TException;
 
+    /**
+     * @return all namespaces
+     * 
+     */
     public List<TNamespaceDescriptor> listNamespaceDescriptors() throws TIOError, org.apache.thrift.TException;
 
   }
@@ -28057,10 +28242,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableDescriptor_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename of the table to get tableDescriptor
+     */
     public TTableName table; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename of the table to get tableDescriptor
+       */
       TABLE((short)1, "table");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -28155,10 +28346,16 @@ public class THBaseService {
       this.table = null;
     }
 
+    /**
+     * the tablename of the table to get tableDescriptor
+     */
     public TTableName getTable() {
       return this.table;
     }
 
+    /**
+     * the tablename of the table to get tableDescriptor
+     */
     public getTableDescriptor_args setTable(TTableName table) {
       this.table = table;
       return this;
@@ -28889,10 +29086,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableDescriptors_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename list of the tables to get tableDescriptor
+     */
     public List<TTableName> tables; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename list of the tables to get tableDescriptor
+       */
       TABLES((short)1, "tables");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -29007,10 +29210,16 @@ public class THBaseService {
       this.tables.add(elem);
     }
 
+    /**
+     * the tablename list of the tables to get tableDescriptor
+     */
     public List<TTableName> getTables() {
       return this.tables;
     }
 
+    /**
+     * the tablename list of the tables to get tableDescriptor
+     */
     public getTableDescriptors_args setTables(List<TTableName> tables) {
       this.tables = tables;
       return this;
@@ -29819,10 +30028,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new tableExists_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename of the tables to check
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename of the tables to check
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -29917,10 +30132,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename of the tables to check
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename of the tables to check
+     */
     public tableExists_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -30654,12 +30875,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableDescriptorsByPattern_argsTupleSchemeFactory());
     }
 
+    /**
+     * The regular expression to match against
+     */
     public String regex; // required
+    /**
+     * set to false if match only against userspace tables
+     */
     public boolean includeSysTables; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * The regular expression to match against
+       */
       REGEX((short)1, "regex"),
+      /**
+       * set to false if match only against userspace tables
+       */
       INCLUDE_SYS_TABLES((short)2, "includeSysTables");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -30767,10 +31000,16 @@ public class THBaseService {
       this.includeSysTables = false;
     }
 
+    /**
+     * The regular expression to match against
+     */
     public String getRegex() {
       return this.regex;
     }
 
+    /**
+     * The regular expression to match against
+     */
     public getTableDescriptorsByPattern_args setRegex(String regex) {
       this.regex = regex;
       return this;
@@ -30791,10 +31030,16 @@ public class THBaseService {
       }
     }
 
+    /**
+     * set to false if match only against userspace tables
+     */
     public boolean isIncludeSysTables() {
       return this.includeSysTables;
     }
 
+    /**
+     * set to false if match only against userspace tables
+     */
     public getTableDescriptorsByPattern_args setIncludeSysTables(boolean includeSysTables) {
       this.includeSysTables = includeSysTables;
       setIncludeSysTablesIsSet(true);
@@ -31629,10 +31874,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableDescriptorsByNamespace_argsTupleSchemeFactory());
     }
 
+    /**
+     * The namesapce's name
+     */
     public String name; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * The namesapce's name
+       */
       NAME((short)1, "name");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -31727,10 +31978,16 @@ public class THBaseService {
       this.name = null;
     }
 
+    /**
+     * The namesapce's name
+     */
     public String getName() {
       return this.name;
     }
 
+    /**
+     * The namesapce's name
+     */
     public getTableDescriptorsByNamespace_args setName(String name) {
       this.name = name;
       return this;
@@ -32506,12 +32763,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableNamesByPattern_argsTupleSchemeFactory());
     }
 
+    /**
+     * The regular expression to match against
+     */
     public String regex; // required
+    /**
+     * set to false if match only against userspace tables
+     */
     public boolean includeSysTables; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * The regular expression to match against
+       */
       REGEX((short)1, "regex"),
+      /**
+       * set to false if match only against userspace tables
+       */
       INCLUDE_SYS_TABLES((short)2, "includeSysTables");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -32619,10 +32888,16 @@ public class THBaseService {
       this.includeSysTables = false;
     }
 
+    /**
+     * The regular expression to match against
+     */
     public String getRegex() {
       return this.regex;
     }
 
+    /**
+     * The regular expression to match against
+     */
     public getTableNamesByPattern_args setRegex(String regex) {
       this.regex = regex;
       return this;
@@ -32643,10 +32918,16 @@ public class THBaseService {
       }
     }
 
+    /**
+     * set to false if match only against userspace tables
+     */
     public boolean isIncludeSysTables() {
       return this.includeSysTables;
     }
 
+    /**
+     * set to false if match only against userspace tables
+     */
     public getTableNamesByPattern_args setIncludeSysTables(boolean includeSysTables) {
       this.includeSysTables = includeSysTables;
       setIncludeSysTablesIsSet(true);
@@ -33481,10 +33762,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getTableNamesByNamespace_argsTupleSchemeFactory());
     }
 
+    /**
+     * The namesapce's name
+     */
     public String name; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * The namesapce's name
+       */
       NAME((short)1, "name");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -33579,10 +33866,16 @@ public class THBaseService {
       this.name = null;
     }
 
+    /**
+     * The namesapce's name
+     */
     public String getName() {
       return this.name;
     }
 
+    /**
+     * The namesapce's name
+     */
     public getTableNamesByNamespace_args setName(String name) {
       this.name = name;
       return this;
@@ -34358,12 +34651,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new createTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * table descriptor for table
+     */
     public TTableDescriptor desc; // required
+    /**
+     * rray of split keys for the initial regions of the table
+     */
     public List<ByteBuffer> splitKeys; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * table descriptor for table
+       */
       DESC((short)1, "desc"),
+      /**
+       * rray of split keys for the initial regions of the table
+       */
       SPLIT_KEYS((short)2, "splitKeys");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -34470,10 +34775,16 @@ public class THBaseService {
       this.splitKeys = null;
     }
 
+    /**
+     * table descriptor for table
+     */
     public TTableDescriptor getDesc() {
       return this.desc;
     }
 
+    /**
+     * table descriptor for table
+     */
     public createTable_args setDesc(TTableDescriptor desc) {
       this.desc = desc;
       return this;
@@ -34509,10 +34820,16 @@ public class THBaseService {
       this.splitKeys.add(elem);
     }
 
+    /**
+     * rray of split keys for the initial regions of the table
+     */
     public List<ByteBuffer> getSplitKeys() {
       return this.splitKeys;
     }
 
+    /**
+     * rray of split keys for the initial regions of the table
+     */
     public createTable_args setSplitKeys(List<ByteBuffer> splitKeys) {
       this.splitKeys = splitKeys;
       return this;
@@ -35229,10 +35546,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new deleteTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to delete
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to delete
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -35327,10 +35650,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to delete
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to delete
+     */
     public deleteTable_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -35952,12 +36281,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new truncateTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to truncate
+     */
     public TTableName tableName; // required
+    /**
+     * whether to  preserve previous splits
+     */
     public boolean preserveSplits; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to truncate
+       */
       TABLE_NAME((short)1, "tableName"),
+      /**
+       * whether to  preserve previous splits
+       */
       PRESERVE_SPLITS((short)2, "preserveSplits");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -36065,10 +36406,16 @@ public class THBaseService {
       this.preserveSplits = false;
     }
 
+    /**
+     * the tablename to truncate
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to truncate
+     */
     public truncateTable_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -36089,10 +36436,16 @@ public class THBaseService {
       }
     }
 
+    /**
+     * whether to  preserve previous splits
+     */
     public boolean isPreserveSplits() {
       return this.preserveSplits;
     }
 
+    /**
+     * whether to  preserve previous splits
+     */
     public truncateTable_args setPreserveSplits(boolean preserveSplits) {
       this.preserveSplits = preserveSplits;
       setPreserveSplitsIsSet(true);
@@ -36773,10 +37126,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new enableTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to enable
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to enable
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -36871,10 +37230,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to enable
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to enable
+     */
     public enableTable_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -37495,10 +37860,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new disableTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to disable
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to disable
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -37593,10 +37964,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to disable
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to disable
+     */
     public disableTable_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -38217,10 +38594,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new isTableEnabled_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to check
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -38315,10 +38698,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to check
+     */
     public isTableEnabled_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -39044,10 +39433,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new isTableDisabled_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to check
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -39142,10 +39537,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to check
+     */
     public isTableDisabled_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -39871,10 +40272,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new isTableAvailable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName tableName; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to check
+       */
       TABLE_NAME((short)1, "tableName");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -39969,10 +40376,16 @@ public class THBaseService {
       this.tableName = null;
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to check
+     */
     public isTableAvailable_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -40699,12 +41112,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new isTableAvailableWithSplit_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName tableName; // required
+    /**
+     * keys to check if the table has been created with all split keys
+     */
     public List<ByteBuffer> splitKeys; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to check
+       */
       TABLE_NAME((short)1, "tableName"),
+      /**
+       * keys to check if the table has been created with all split keys
+       */
       SPLIT_KEYS((short)2, "splitKeys");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -40811,10 +41236,16 @@ public class THBaseService {
       this.splitKeys = null;
     }
 
+    /**
+     * the tablename to check
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to check
+     */
     public isTableAvailableWithSplit_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -40850,10 +41281,16 @@ public class THBaseService {
       this.splitKeys.add(elem);
     }
 
+    /**
+     * keys to check if the table has been created with all split keys
+     */
     public List<ByteBuffer> getSplitKeys() {
       return this.splitKeys;
     }
 
+    /**
+     * keys to check if the table has been created with all split keys
+     */
     public isTableAvailableWithSplit_args setSplitKeys(List<ByteBuffer> splitKeys) {
       this.splitKeys = splitKeys;
       return this;
@@ -41676,12 +42113,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new addColumnFamily_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to add column family to
+     */
     public TTableName tableName; // required
+    /**
+     * column family descriptor of column family to be added
+     */
     public TColumnFamilyDescriptor column; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to add column family to
+       */
       TABLE_NAME((short)1, "tableName"),
+      /**
+       * column family descriptor of column family to be added
+       */
       COLUMN((short)2, "column");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -41786,10 +42235,16 @@ public class THBaseService {
       this.column = null;
     }
 
+    /**
+     * the tablename to add column family to
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to add column family to
+     */
     public addColumnFamily_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -41810,10 +42265,16 @@ public class THBaseService {
       }
     }
 
+    /**
+     * column family descriptor of column family to be added
+     */
     public TColumnFamilyDescriptor getColumn() {
       return this.column;
     }
 
+    /**
+     * column family descriptor of column family to be added
+     */
     public addColumnFamily_args setColumn(TColumnFamilyDescriptor column) {
       this.column = column;
       return this;
@@ -42504,12 +42965,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new deleteColumnFamily_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to delete column family from
+     */
     public TTableName tableName; // required
+    /**
+     * name of column family to be deleted
+     */
     public ByteBuffer column; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to delete column family from
+       */
       TABLE_NAME((short)1, "tableName"),
+      /**
+       * name of column family to be deleted
+       */
       COLUMN((short)2, "column");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -42614,10 +43087,16 @@ public class THBaseService {
       this.column = null;
     }
 
+    /**
+     * the tablename to delete column family from
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to delete column family from
+     */
     public deleteColumnFamily_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -42638,6 +43117,9 @@ public class THBaseService {
       }
     }
 
+    /**
+     * name of column family to be deleted
+     */
     public byte[] getColumn() {
       setColumn(org.apache.thrift.TBaseHelper.rightSize(column));
       return column == null ? null : column.array();
@@ -42647,6 +43129,9 @@ public class THBaseService {
       return org.apache.thrift.TBaseHelper.copyBinary(column);
     }
 
+    /**
+     * name of column family to be deleted
+     */
     public deleteColumnFamily_args setColumn(byte[] column) {
       this.column = column == null ? (ByteBuffer)null : ByteBuffer.wrap(Arrays.copyOf(column, column.length));
       return this;
@@ -43337,12 +43822,24 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new modifyColumnFamily_argsTupleSchemeFactory());
     }
 
+    /**
+     * the tablename to modify column family
+     */
     public TTableName tableName; // required
+    /**
+     * column family descriptor of column family to be modified
+     */
     public TColumnFamilyDescriptor column; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the tablename to modify column family
+       */
       TABLE_NAME((short)1, "tableName"),
+      /**
+       * column family descriptor of column family to be modified
+       */
       COLUMN((short)2, "column");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -43447,10 +43944,16 @@ public class THBaseService {
       this.column = null;
     }
 
+    /**
+     * the tablename to modify column family
+     */
     public TTableName getTableName() {
       return this.tableName;
     }
 
+    /**
+     * the tablename to modify column family
+     */
     public modifyColumnFamily_args setTableName(TTableName tableName) {
       this.tableName = tableName;
       return this;
@@ -43471,10 +43974,16 @@ public class THBaseService {
       }
     }
 
+    /**
+     * column family descriptor of column family to be modified
+     */
     public TColumnFamilyDescriptor getColumn() {
       return this.column;
     }
 
+    /**
+     * column family descriptor of column family to be modified
+     */
     public modifyColumnFamily_args setColumn(TColumnFamilyDescriptor column) {
       this.column = column;
       return this;
@@ -44164,10 +44673,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new modifyTable_argsTupleSchemeFactory());
     }
 
+    /**
+     * the descriptor of the table to modify
+     */
     public TTableDescriptor desc; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * the descriptor of the table to modify
+       */
       DESC((short)1, "desc");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -44262,10 +44777,16 @@ public class THBaseService {
       this.desc = null;
     }
 
+    /**
+     * the descriptor of the table to modify
+     */
     public TTableDescriptor getDesc() {
       return this.desc;
     }
 
+    /**
+     * the descriptor of the table to modify
+     */
     public modifyTable_args setDesc(TTableDescriptor desc) {
       this.desc = desc;
       return this;
@@ -44886,10 +45407,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new createNamespace_argsTupleSchemeFactory());
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public TNamespaceDescriptor namespaceDesc; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * descriptor which describes the new namespace
+       */
       NAMESPACE_DESC((short)1, "namespaceDesc");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -44984,10 +45511,16 @@ public class THBaseService {
       this.namespaceDesc = null;
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public TNamespaceDescriptor getNamespaceDesc() {
       return this.namespaceDesc;
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public createNamespace_args setNamespaceDesc(TNamespaceDescriptor namespaceDesc) {
       this.namespaceDesc = namespaceDesc;
       return this;
@@ -45608,10 +46141,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new modifyNamespace_argsTupleSchemeFactory());
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public TNamespaceDescriptor namespaceDesc; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * descriptor which describes the new namespace
+       */
       NAMESPACE_DESC((short)1, "namespaceDesc");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -45706,10 +46245,16 @@ public class THBaseService {
       this.namespaceDesc = null;
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public TNamespaceDescriptor getNamespaceDesc() {
       return this.namespaceDesc;
     }
 
+    /**
+     * descriptor which describes the new namespace
+     */
     public modifyNamespace_args setNamespaceDesc(TNamespaceDescriptor namespaceDesc) {
       this.namespaceDesc = namespaceDesc;
       return this;
@@ -46330,10 +46875,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new deleteNamespace_argsTupleSchemeFactory());
     }
 
+    /**
+     * namespace name
+     */
     public String name; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * namespace name
+       */
       NAME((short)1, "name");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -46428,10 +46979,16 @@ public class THBaseService {
       this.name = null;
     }
 
+    /**
+     * namespace name
+     */
     public String getName() {
       return this.name;
     }
 
+    /**
+     * namespace name
+     */
     public deleteNamespace_args setName(String name) {
       this.name = name;
       return this;
@@ -47047,10 +47604,16 @@ public class THBaseService {
       schemes.put(TupleScheme.class, new getNamespaceDescriptor_argsTupleSchemeFactory());
     }
 
+    /**
+     * name of namespace descriptor
+     */
     public String name; // required
 
     /** The set of fields this struct contains, along with convenience methods for finding and manipulating them. */
     public enum _Fields implements org.apache.thrift.TFieldIdEnum {
+      /**
+       * name of namespace descriptor
+       */
       NAME((short)1, "name");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
@@ -47145,10 +47708,16 @@ public class THBaseService {
       this.name = null;
     }
 
+    /**
+     * name of namespace descriptor
+     */
     public String getName() {
       return this.name;
     }
 
+    /**
+     * name of namespace descriptor
+     */
     public getNamespaceDescriptor_args setName(String name) {
       this.name = name;
       return this;

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionInfo.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionInfo.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionInfo.java
index 8450f5b..b430d4d 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionInfo.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionInfo.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class THRegionInfo implements org.apache.thrift.TBase<THRegionInfo, THRegionInfo._Fields>, java.io.Serializable, Cloneable, Comparable<THRegionInfo> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("THRegionInfo");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionLocation.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionLocation.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionLocation.java
index b1146e9..f64951e 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionLocation.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/THRegionLocation.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class THRegionLocation implements org.apache.thrift.TBase<THRegionLocation, THRegionLocation._Fields>, java.io.Serializable, Cloneable, Comparable<THRegionLocation> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("THRegionLocation");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIOError.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIOError.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIOError.java
index 9569c3f..ccb8514 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIOError.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIOError.java
@@ -39,7 +39,7 @@ import org.slf4j.LoggerFactory;
  * to the HBase master or a HBase region server. Also used to return
  * more general HBase error conditions.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TIOError extends TException implements org.apache.thrift.TBase<TIOError, TIOError._Fields>, java.io.Serializable, Cloneable, Comparable<TIOError> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TIOError");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIllegalArgument.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIllegalArgument.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIllegalArgument.java
index 6734dec..4436fc0 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIllegalArgument.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIllegalArgument.java
@@ -38,7 +38,7 @@ import org.slf4j.LoggerFactory;
  * A TIllegalArgument exception indicates an illegal or invalid
  * argument was passed into a procedure.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TIllegalArgument extends TException implements org.apache.thrift.TBase<TIllegalArgument, TIllegalArgument._Fields>, java.io.Serializable, Cloneable, Comparable<TIllegalArgument> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TIllegalArgument");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIncrement.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIncrement.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIncrement.java
index 3663e61..70d8eab 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIncrement.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TIncrement.java
@@ -41,7 +41,7 @@ import org.slf4j.LoggerFactory;
  * by changing the durability. If you don't provide durability, it defaults to
  * column family's default setting for durability.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TIncrement implements org.apache.thrift.TBase<TIncrement, TIncrement._Fields>, java.io.Serializable, Cloneable, Comparable<TIncrement> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TIncrement");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TKeepDeletedCells.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TKeepDeletedCells.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TKeepDeletedCells.java
index 4ce64e2..24f4d09 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TKeepDeletedCells.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TKeepDeletedCells.java
@@ -11,6 +11,10 @@ import java.util.Map;
 import java.util.HashMap;
 import org.apache.thrift.TEnum;
 
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.KeepDeletedCells
+ */
 public enum TKeepDeletedCells implements org.apache.thrift.TEnum {
   /**
    * Deleted Cells are not retained.

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TNamespaceDescriptor.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TNamespaceDescriptor.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TNamespaceDescriptor.java
index 72d4916..1c0b41b 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TNamespaceDescriptor.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TNamespaceDescriptor.java
@@ -34,7 +34,11 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+/**
+ * Thrift wrapper around
+ * org.apache.hadoop.hbase.NamespaceDescriptor
+ */
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TNamespaceDescriptor implements org.apache.thrift.TBase<TNamespaceDescriptor, TNamespaceDescriptor._Fields>, java.io.Serializable, Cloneable, Comparable<TNamespaceDescriptor> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TNamespaceDescriptor");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TPut.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TPut.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TPut.java
index ad82c67..5739d78 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TPut.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TPut.java
@@ -46,7 +46,7 @@ import org.slf4j.LoggerFactory;
  * by changing the durability. If you don't provide durability, it defaults to
  * column family's default setting for durability.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TPut implements org.apache.thrift.TBase<TPut, TPut._Fields>, java.io.Serializable, Cloneable, Comparable<TPut> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TPut");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TResult.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TResult.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TResult.java
index c19cf89..3d40ea2 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TResult.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TResult.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * if no Result is found, row and columnValues will not be set.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TResult implements org.apache.thrift.TBase<TResult, TResult._Fields>, java.io.Serializable, Cloneable, Comparable<TResult> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TResult");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TRowMutations.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TRowMutations.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TRowMutations.java
index a3ab58e..cc846f5 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TRowMutations.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TRowMutations.java
@@ -37,7 +37,7 @@ import org.slf4j.LoggerFactory;
 /**
  * A TRowMutations object is used to apply a number of Mutations to a single row.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TRowMutations implements org.apache.thrift.TBase<TRowMutations, TRowMutations._Fields>, java.io.Serializable, Cloneable, Comparable<TRowMutations> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TRowMutations");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TScan.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TScan.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TScan.java
index ddcedb4..99046cc 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TScan.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TScan.java
@@ -38,7 +38,7 @@ import org.slf4j.LoggerFactory;
  * Any timestamps in the columns are ignored but the colFamTimeRangeMap included, use timeRange to select by timestamp.
  * Max versions defaults to 1.
  */
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TScan implements org.apache.thrift.TBase<TScan, TScan._Fields>, java.io.Serializable, Cloneable, Comparable<TScan> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TScan");
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/b620334c/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TServerName.java
----------------------------------------------------------------------
diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TServerName.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TServerName.java
index 65f963d..e7e0372 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TServerName.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TServerName.java
@@ -34,7 +34,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 @SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
-@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-27")
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2018-12-28")
 public class TServerName implements org.apache.thrift.TBase<TServerName, TServerName._Fields>, java.io.Serializable, Cloneable, Comparable<TServerName> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TServerName");