You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by sr...@apache.org on 2016/06/01 22:06:06 UTC

[10/11] sentry git commit: SENTRY-1236: Bump thrift version to 0.9.3 (Sravya Tirukkovalur, Reviewed by: Lenni Kuff and Hao Hao)

http://git-wip-us.apache.org/repos/asf/sentry/blob/e67bde0d/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsDump.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsDump.java b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsDump.java
index caf9ad1..c1b42a0 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsDump.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsDump.java
@@ -1,12 +1,11 @@
 /**
- * Autogenerated by Thrift Compiler (0.9.0)
+ * Autogenerated by Thrift Compiler (0.9.3)
  *
  * DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
  *  @generated
  */
 package org.apache.sentry.hdfs.service.thrift;
 
-import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.thrift.scheme.IScheme;
 import org.apache.thrift.scheme.SchemeFactory;
 import org.apache.thrift.scheme.StandardScheme;
@@ -16,6 +15,8 @@ import org.apache.thrift.protocol.TTupleProtocol;
 import org.apache.thrift.protocol.TProtocolException;
 import org.apache.thrift.EncodingUtils;
 import org.apache.thrift.TException;
+import org.apache.thrift.async.AsyncMethodCallback;
+import org.apache.thrift.server.AbstractNonblockingServer.*;
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Map;
@@ -28,10 +29,13 @@ import java.util.Collections;
 import java.util.BitSet;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
+import javax.annotation.Generated;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDump._Fields>, java.io.Serializable, Cloneable {
+@SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2016-05-05")
+public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDump._Fields>, java.io.Serializable, Cloneable, Comparable<TPathsDump> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TPathsDump");
 
   private static final org.apache.thrift.protocol.TField ROOT_ID_FIELD_DESC = new org.apache.thrift.protocol.TField("rootId", org.apache.thrift.protocol.TType.I32, (short)1);
@@ -143,7 +147,7 @@ public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDum
     __isset_bitfield = other.__isset_bitfield;
     this.rootId = other.rootId;
     if (other.isSetNodeMap()) {
-      Map<Integer,TPathEntry> __this__nodeMap = new HashMap<Integer,TPathEntry>();
+      Map<Integer,TPathEntry> __this__nodeMap = new HashMap<Integer,TPathEntry>(other.nodeMap.size());
       for (Map.Entry<Integer, TPathEntry> other_element : other.nodeMap.entrySet()) {
 
         Integer other_element_key = other_element.getKey();
@@ -250,7 +254,7 @@ public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDum
   public Object getFieldValue(_Fields field) {
     switch (field) {
     case ROOT_ID:
-      return Integer.valueOf(getRootId());
+      return getRootId();
 
     case NODE_MAP:
       return getNodeMap();
@@ -310,45 +314,45 @@ public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDum
 
   @Override
   public int hashCode() {
-    HashCodeBuilder builder = new HashCodeBuilder();
+    List<Object> list = new ArrayList<Object>();
 
     boolean present_rootId = true;
-    builder.append(present_rootId);
+    list.add(present_rootId);
     if (present_rootId)
-      builder.append(rootId);
+      list.add(rootId);
 
     boolean present_nodeMap = true && (isSetNodeMap());
-    builder.append(present_nodeMap);
+    list.add(present_nodeMap);
     if (present_nodeMap)
-      builder.append(nodeMap);
+      list.add(nodeMap);
 
-    return builder.toHashCode();
+    return list.hashCode();
   }
 
+  @Override
   public int compareTo(TPathsDump other) {
     if (!getClass().equals(other.getClass())) {
       return getClass().getName().compareTo(other.getClass().getName());
     }
 
     int lastComparison = 0;
-    TPathsDump typedOther = (TPathsDump)other;
 
-    lastComparison = Boolean.valueOf(isSetRootId()).compareTo(typedOther.isSetRootId());
+    lastComparison = Boolean.valueOf(isSetRootId()).compareTo(other.isSetRootId());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetRootId()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.rootId, typedOther.rootId);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.rootId, other.rootId);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetNodeMap()).compareTo(typedOther.isSetNodeMap());
+    lastComparison = Boolean.valueOf(isSetNodeMap()).compareTo(other.isSetNodeMap());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetNodeMap()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.nodeMap, typedOther.nodeMap);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.nodeMap, other.nodeMap);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -450,14 +454,14 @@ public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDum
               {
                 org.apache.thrift.protocol.TMap _map48 = iprot.readMapBegin();
                 struct.nodeMap = new HashMap<Integer,TPathEntry>(2*_map48.size);
-                for (int _i49 = 0; _i49 < _map48.size; ++_i49)
+                int _key49;
+                TPathEntry _val50;
+                for (int _i51 = 0; _i51 < _map48.size; ++_i51)
                 {
-                  int _key50; // required
-                  TPathEntry _val51; // required
-                  _key50 = iprot.readI32();
-                  _val51 = new TPathEntry();
-                  _val51.read(iprot);
-                  struct.nodeMap.put(_key50, _val51);
+                  _key49 = iprot.readI32();
+                  _val50 = new TPathEntry();
+                  _val50.read(iprot);
+                  struct.nodeMap.put(_key49, _val50);
                 }
                 iprot.readMapEnd();
               }
@@ -531,14 +535,14 @@ public class TPathsDump implements org.apache.thrift.TBase<TPathsDump, TPathsDum
       {
         org.apache.thrift.protocol.TMap _map54 = new org.apache.thrift.protocol.TMap(org.apache.thrift.protocol.TType.I32, org.apache.thrift.protocol.TType.STRUCT, iprot.readI32());
         struct.nodeMap = new HashMap<Integer,TPathEntry>(2*_map54.size);
-        for (int _i55 = 0; _i55 < _map54.size; ++_i55)
+        int _key55;
+        TPathEntry _val56;
+        for (int _i57 = 0; _i57 < _map54.size; ++_i57)
         {
-          int _key56; // required
-          TPathEntry _val57; // required
-          _key56 = iprot.readI32();
-          _val57 = new TPathEntry();
-          _val57.read(iprot);
-          struct.nodeMap.put(_key56, _val57);
+          _key55 = iprot.readI32();
+          _val56 = new TPathEntry();
+          _val56.read(iprot);
+          struct.nodeMap.put(_key55, _val56);
         }
       }
       struct.setNodeMapIsSet(true);

http://git-wip-us.apache.org/repos/asf/sentry/blob/e67bde0d/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java
index f52b4ff..f469d4a 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java
@@ -1,12 +1,11 @@
 /**
- * Autogenerated by Thrift Compiler (0.9.0)
+ * Autogenerated by Thrift Compiler (0.9.3)
  *
  * DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
  *  @generated
  */
 package org.apache.sentry.hdfs.service.thrift;
 
-import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.thrift.scheme.IScheme;
 import org.apache.thrift.scheme.SchemeFactory;
 import org.apache.thrift.scheme.StandardScheme;
@@ -16,6 +15,8 @@ import org.apache.thrift.protocol.TTupleProtocol;
 import org.apache.thrift.protocol.TProtocolException;
 import org.apache.thrift.EncodingUtils;
 import org.apache.thrift.TException;
+import org.apache.thrift.async.AsyncMethodCallback;
+import org.apache.thrift.server.AbstractNonblockingServer.*;
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Map;
@@ -28,10 +29,13 @@ import java.util.Collections;
 import java.util.BitSet;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
+import javax.annotation.Generated;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPathsUpdate._Fields>, java.io.Serializable, Cloneable {
+@SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2016-05-05")
+public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPathsUpdate._Fields>, java.io.Serializable, Cloneable, Comparable<TPathsUpdate> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TPathsUpdate");
 
   private static final org.apache.thrift.protocol.TField HAS_FULL_IMAGE_FIELD_DESC = new org.apache.thrift.protocol.TField("hasFullImage", org.apache.thrift.protocol.TType.BOOL, (short)1);
@@ -121,7 +125,7 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
   private static final int __HASFULLIMAGE_ISSET_ID = 0;
   private static final int __SEQNUM_ISSET_ID = 1;
   private byte __isset_bitfield = 0;
-  private _Fields optionals[] = {_Fields.PATHS_DUMP};
+  private static final _Fields optionals[] = {_Fields.PATHS_DUMP};
   public static final Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> metaDataMap;
   static {
     Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
@@ -165,7 +169,7 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
     }
     this.seqNum = other.seqNum;
     if (other.isSetPathChanges()) {
-      List<TPathChanges> __this__pathChanges = new ArrayList<TPathChanges>();
+      List<TPathChanges> __this__pathChanges = new ArrayList<TPathChanges>(other.pathChanges.size());
       for (TPathChanges other_element : other.pathChanges) {
         __this__pathChanges.add(new TPathChanges(other_element));
       }
@@ -332,13 +336,13 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
   public Object getFieldValue(_Fields field) {
     switch (field) {
     case HAS_FULL_IMAGE:
-      return Boolean.valueOf(isHasFullImage());
+      return isHasFullImage();
 
     case PATHS_DUMP:
       return getPathsDump();
 
     case SEQ_NUM:
-      return Long.valueOf(getSeqNum());
+      return getSeqNum();
 
     case PATH_CHANGES:
       return getPathChanges();
@@ -420,75 +424,75 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
 
   @Override
   public int hashCode() {
-    HashCodeBuilder builder = new HashCodeBuilder();
+    List<Object> list = new ArrayList<Object>();
 
     boolean present_hasFullImage = true;
-    builder.append(present_hasFullImage);
+    list.add(present_hasFullImage);
     if (present_hasFullImage)
-      builder.append(hasFullImage);
+      list.add(hasFullImage);
 
     boolean present_pathsDump = true && (isSetPathsDump());
-    builder.append(present_pathsDump);
+    list.add(present_pathsDump);
     if (present_pathsDump)
-      builder.append(pathsDump);
+      list.add(pathsDump);
 
     boolean present_seqNum = true;
-    builder.append(present_seqNum);
+    list.add(present_seqNum);
     if (present_seqNum)
-      builder.append(seqNum);
+      list.add(seqNum);
 
     boolean present_pathChanges = true && (isSetPathChanges());
-    builder.append(present_pathChanges);
+    list.add(present_pathChanges);
     if (present_pathChanges)
-      builder.append(pathChanges);
+      list.add(pathChanges);
 
-    return builder.toHashCode();
+    return list.hashCode();
   }
 
+  @Override
   public int compareTo(TPathsUpdate other) {
     if (!getClass().equals(other.getClass())) {
       return getClass().getName().compareTo(other.getClass().getName());
     }
 
     int lastComparison = 0;
-    TPathsUpdate typedOther = (TPathsUpdate)other;
 
-    lastComparison = Boolean.valueOf(isSetHasFullImage()).compareTo(typedOther.isSetHasFullImage());
+    lastComparison = Boolean.valueOf(isSetHasFullImage()).compareTo(other.isSetHasFullImage());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetHasFullImage()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.hasFullImage, typedOther.hasFullImage);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.hasFullImage, other.hasFullImage);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetPathsDump()).compareTo(typedOther.isSetPathsDump());
+    lastComparison = Boolean.valueOf(isSetPathsDump()).compareTo(other.isSetPathsDump());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetPathsDump()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.pathsDump, typedOther.pathsDump);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.pathsDump, other.pathsDump);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetSeqNum()).compareTo(typedOther.isSetSeqNum());
+    lastComparison = Boolean.valueOf(isSetSeqNum()).compareTo(other.isSetSeqNum());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetSeqNum()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.seqNum, typedOther.seqNum);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.seqNum, other.seqNum);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetPathChanges()).compareTo(typedOther.isSetPathChanges());
+    lastComparison = Boolean.valueOf(isSetPathChanges()).compareTo(other.isSetPathChanges());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetPathChanges()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.pathChanges, typedOther.pathChanges);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.pathChanges, other.pathChanges);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -628,12 +632,12 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
               {
                 org.apache.thrift.protocol.TList _list58 = iprot.readListBegin();
                 struct.pathChanges = new ArrayList<TPathChanges>(_list58.size);
-                for (int _i59 = 0; _i59 < _list58.size; ++_i59)
+                TPathChanges _elem59;
+                for (int _i60 = 0; _i60 < _list58.size; ++_i60)
                 {
-                  TPathChanges _elem60; // required
-                  _elem60 = new TPathChanges();
-                  _elem60.read(iprot);
-                  struct.pathChanges.add(_elem60);
+                  _elem59 = new TPathChanges();
+                  _elem59.read(iprot);
+                  struct.pathChanges.add(_elem59);
                 }
                 iprot.readListEnd();
               }
@@ -726,12 +730,12 @@ public class TPathsUpdate implements org.apache.thrift.TBase<TPathsUpdate, TPath
       {
         org.apache.thrift.protocol.TList _list63 = new org.apache.thrift.protocol.TList(org.apache.thrift.protocol.TType.STRUCT, iprot.readI32());
         struct.pathChanges = new ArrayList<TPathChanges>(_list63.size);
-        for (int _i64 = 0; _i64 < _list63.size; ++_i64)
+        TPathChanges _elem64;
+        for (int _i65 = 0; _i65 < _list63.size; ++_i65)
         {
-          TPathChanges _elem65; // required
-          _elem65 = new TPathChanges();
-          _elem65.read(iprot);
-          struct.pathChanges.add(_elem65);
+          _elem64 = new TPathChanges();
+          _elem64.read(iprot);
+          struct.pathChanges.add(_elem64);
         }
       }
       struct.setPathChangesIsSet(true);

http://git-wip-us.apache.org/repos/asf/sentry/blob/e67bde0d/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPermissionsUpdate.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPermissionsUpdate.java b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPermissionsUpdate.java
index bfb6cbc..727b705 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPermissionsUpdate.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPermissionsUpdate.java
@@ -1,12 +1,11 @@
 /**
- * Autogenerated by Thrift Compiler (0.9.0)
+ * Autogenerated by Thrift Compiler (0.9.3)
  *
  * DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
  *  @generated
  */
 package org.apache.sentry.hdfs.service.thrift;
 
-import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.thrift.scheme.IScheme;
 import org.apache.thrift.scheme.SchemeFactory;
 import org.apache.thrift.scheme.StandardScheme;
@@ -16,6 +15,8 @@ import org.apache.thrift.protocol.TTupleProtocol;
 import org.apache.thrift.protocol.TProtocolException;
 import org.apache.thrift.EncodingUtils;
 import org.apache.thrift.TException;
+import org.apache.thrift.async.AsyncMethodCallback;
+import org.apache.thrift.server.AbstractNonblockingServer.*;
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Map;
@@ -28,10 +29,13 @@ import java.util.Collections;
 import java.util.BitSet;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
+import javax.annotation.Generated;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsUpdate, TPermissionsUpdate._Fields>, java.io.Serializable, Cloneable {
+@SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2016-05-05")
+public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsUpdate, TPermissionsUpdate._Fields>, java.io.Serializable, Cloneable, Comparable<TPermissionsUpdate> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TPermissionsUpdate");
 
   private static final org.apache.thrift.protocol.TField HASFULL_IMAGE_FIELD_DESC = new org.apache.thrift.protocol.TField("hasfullImage", org.apache.thrift.protocol.TType.BOOL, (short)1);
@@ -166,7 +170,7 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
     this.hasfullImage = other.hasfullImage;
     this.seqNum = other.seqNum;
     if (other.isSetPrivilegeChanges()) {
-      Map<String,TPrivilegeChanges> __this__privilegeChanges = new HashMap<String,TPrivilegeChanges>();
+      Map<String,TPrivilegeChanges> __this__privilegeChanges = new HashMap<String,TPrivilegeChanges>(other.privilegeChanges.size());
       for (Map.Entry<String, TPrivilegeChanges> other_element : other.privilegeChanges.entrySet()) {
 
         String other_element_key = other_element.getKey();
@@ -181,7 +185,7 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
       this.privilegeChanges = __this__privilegeChanges;
     }
     if (other.isSetRoleChanges()) {
-      Map<String,TRoleChanges> __this__roleChanges = new HashMap<String,TRoleChanges>();
+      Map<String,TRoleChanges> __this__roleChanges = new HashMap<String,TRoleChanges>(other.roleChanges.size());
       for (Map.Entry<String, TRoleChanges> other_element : other.roleChanges.entrySet()) {
 
         String other_element_key = other_element.getKey();
@@ -363,10 +367,10 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
   public Object getFieldValue(_Fields field) {
     switch (field) {
     case HASFULL_IMAGE:
-      return Boolean.valueOf(isHasfullImage());
+      return isHasfullImage();
 
     case SEQ_NUM:
-      return Long.valueOf(getSeqNum());
+      return getSeqNum();
 
     case PRIVILEGE_CHANGES:
       return getPrivilegeChanges();
@@ -451,75 +455,75 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
 
   @Override
   public int hashCode() {
-    HashCodeBuilder builder = new HashCodeBuilder();
+    List<Object> list = new ArrayList<Object>();
 
     boolean present_hasfullImage = true;
-    builder.append(present_hasfullImage);
+    list.add(present_hasfullImage);
     if (present_hasfullImage)
-      builder.append(hasfullImage);
+      list.add(hasfullImage);
 
     boolean present_seqNum = true;
-    builder.append(present_seqNum);
+    list.add(present_seqNum);
     if (present_seqNum)
-      builder.append(seqNum);
+      list.add(seqNum);
 
     boolean present_privilegeChanges = true && (isSetPrivilegeChanges());
-    builder.append(present_privilegeChanges);
+    list.add(present_privilegeChanges);
     if (present_privilegeChanges)
-      builder.append(privilegeChanges);
+      list.add(privilegeChanges);
 
     boolean present_roleChanges = true && (isSetRoleChanges());
-    builder.append(present_roleChanges);
+    list.add(present_roleChanges);
     if (present_roleChanges)
-      builder.append(roleChanges);
+      list.add(roleChanges);
 
-    return builder.toHashCode();
+    return list.hashCode();
   }
 
+  @Override
   public int compareTo(TPermissionsUpdate other) {
     if (!getClass().equals(other.getClass())) {
       return getClass().getName().compareTo(other.getClass().getName());
     }
 
     int lastComparison = 0;
-    TPermissionsUpdate typedOther = (TPermissionsUpdate)other;
 
-    lastComparison = Boolean.valueOf(isSetHasfullImage()).compareTo(typedOther.isSetHasfullImage());
+    lastComparison = Boolean.valueOf(isSetHasfullImage()).compareTo(other.isSetHasfullImage());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetHasfullImage()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.hasfullImage, typedOther.hasfullImage);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.hasfullImage, other.hasfullImage);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetSeqNum()).compareTo(typedOther.isSetSeqNum());
+    lastComparison = Boolean.valueOf(isSetSeqNum()).compareTo(other.isSetSeqNum());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetSeqNum()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.seqNum, typedOther.seqNum);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.seqNum, other.seqNum);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetPrivilegeChanges()).compareTo(typedOther.isSetPrivilegeChanges());
+    lastComparison = Boolean.valueOf(isSetPrivilegeChanges()).compareTo(other.isSetPrivilegeChanges());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetPrivilegeChanges()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.privilegeChanges, typedOther.privilegeChanges);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.privilegeChanges, other.privilegeChanges);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetRoleChanges()).compareTo(typedOther.isSetRoleChanges());
+    lastComparison = Boolean.valueOf(isSetRoleChanges()).compareTo(other.isSetRoleChanges());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetRoleChanges()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.roleChanges, typedOther.roleChanges);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.roleChanges, other.roleChanges);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -649,14 +653,14 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
               {
                 org.apache.thrift.protocol.TMap _map102 = iprot.readMapBegin();
                 struct.privilegeChanges = new HashMap<String,TPrivilegeChanges>(2*_map102.size);
-                for (int _i103 = 0; _i103 < _map102.size; ++_i103)
+                String _key103;
+                TPrivilegeChanges _val104;
+                for (int _i105 = 0; _i105 < _map102.size; ++_i105)
                 {
-                  String _key104; // required
-                  TPrivilegeChanges _val105; // required
-                  _key104 = iprot.readString();
-                  _val105 = new TPrivilegeChanges();
-                  _val105.read(iprot);
-                  struct.privilegeChanges.put(_key104, _val105);
+                  _key103 = iprot.readString();
+                  _val104 = new TPrivilegeChanges();
+                  _val104.read(iprot);
+                  struct.privilegeChanges.put(_key103, _val104);
                 }
                 iprot.readMapEnd();
               }
@@ -670,14 +674,14 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
               {
                 org.apache.thrift.protocol.TMap _map106 = iprot.readMapBegin();
                 struct.roleChanges = new HashMap<String,TRoleChanges>(2*_map106.size);
-                for (int _i107 = 0; _i107 < _map106.size; ++_i107)
+                String _key107;
+                TRoleChanges _val108;
+                for (int _i109 = 0; _i109 < _map106.size; ++_i109)
                 {
-                  String _key108; // required
-                  TRoleChanges _val109; // required
-                  _key108 = iprot.readString();
-                  _val109 = new TRoleChanges();
-                  _val109.read(iprot);
-                  struct.roleChanges.put(_key108, _val109);
+                  _key107 = iprot.readString();
+                  _val108 = new TRoleChanges();
+                  _val108.read(iprot);
+                  struct.roleChanges.put(_key107, _val108);
                 }
                 iprot.readMapEnd();
               }
@@ -778,28 +782,28 @@ public class TPermissionsUpdate implements org.apache.thrift.TBase<TPermissionsU
       {
         org.apache.thrift.protocol.TMap _map114 = new org.apache.thrift.protocol.TMap(org.apache.thrift.protocol.TType.STRING, org.apache.thrift.protocol.TType.STRUCT, iprot.readI32());
         struct.privilegeChanges = new HashMap<String,TPrivilegeChanges>(2*_map114.size);
-        for (int _i115 = 0; _i115 < _map114.size; ++_i115)
+        String _key115;
+        TPrivilegeChanges _val116;
+        for (int _i117 = 0; _i117 < _map114.size; ++_i117)
         {
-          String _key116; // required
-          TPrivilegeChanges _val117; // required
-          _key116 = iprot.readString();
-          _val117 = new TPrivilegeChanges();
-          _val117.read(iprot);
-          struct.privilegeChanges.put(_key116, _val117);
+          _key115 = iprot.readString();
+          _val116 = new TPrivilegeChanges();
+          _val116.read(iprot);
+          struct.privilegeChanges.put(_key115, _val116);
         }
       }
       struct.setPrivilegeChangesIsSet(true);
       {
         org.apache.thrift.protocol.TMap _map118 = new org.apache.thrift.protocol.TMap(org.apache.thrift.protocol.TType.STRING, org.apache.thrift.protocol.TType.STRUCT, iprot.readI32());
         struct.roleChanges = new HashMap<String,TRoleChanges>(2*_map118.size);
-        for (int _i119 = 0; _i119 < _map118.size; ++_i119)
+        String _key119;
+        TRoleChanges _val120;
+        for (int _i121 = 0; _i121 < _map118.size; ++_i121)
         {
-          String _key120; // required
-          TRoleChanges _val121; // required
-          _key120 = iprot.readString();
-          _val121 = new TRoleChanges();
-          _val121.read(iprot);
-          struct.roleChanges.put(_key120, _val121);
+          _key119 = iprot.readString();
+          _val120 = new TRoleChanges();
+          _val120.read(iprot);
+          struct.roleChanges.put(_key119, _val120);
         }
       }
       struct.setRoleChangesIsSet(true);

http://git-wip-us.apache.org/repos/asf/sentry/blob/e67bde0d/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegeChanges.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegeChanges.java b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegeChanges.java
index dc4626e..277118d 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegeChanges.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegeChanges.java
@@ -1,12 +1,11 @@
 /**
- * Autogenerated by Thrift Compiler (0.9.0)
+ * Autogenerated by Thrift Compiler (0.9.3)
  *
  * DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
  *  @generated
  */
 package org.apache.sentry.hdfs.service.thrift;
 
-import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.thrift.scheme.IScheme;
 import org.apache.thrift.scheme.SchemeFactory;
 import org.apache.thrift.scheme.StandardScheme;
@@ -16,6 +15,8 @@ import org.apache.thrift.protocol.TTupleProtocol;
 import org.apache.thrift.protocol.TProtocolException;
 import org.apache.thrift.EncodingUtils;
 import org.apache.thrift.TException;
+import org.apache.thrift.async.AsyncMethodCallback;
+import org.apache.thrift.server.AbstractNonblockingServer.*;
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Map;
@@ -28,10 +29,13 @@ import java.util.Collections;
 import java.util.BitSet;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
+import javax.annotation.Generated;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChanges, TPrivilegeChanges._Fields>, java.io.Serializable, Cloneable {
+@SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2016-05-05")
+public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChanges, TPrivilegeChanges._Fields>, java.io.Serializable, Cloneable, Comparable<TPrivilegeChanges> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TPrivilegeChanges");
 
   private static final org.apache.thrift.protocol.TField AUTHZ_OBJ_FIELD_DESC = new org.apache.thrift.protocol.TField("authzObj", org.apache.thrift.protocol.TType.STRING, (short)1);
@@ -152,33 +156,11 @@ public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChan
       this.authzObj = other.authzObj;
     }
     if (other.isSetAddPrivileges()) {
-      Map<String,String> __this__addPrivileges = new HashMap<String,String>();
-      for (Map.Entry<String, String> other_element : other.addPrivileges.entrySet()) {
-
-        String other_element_key = other_element.getKey();
-        String other_element_value = other_element.getValue();
-
-        String __this__addPrivileges_copy_key = other_element_key;
-
-        String __this__addPrivileges_copy_value = other_element_value;
-
-        __this__addPrivileges.put(__this__addPrivileges_copy_key, __this__addPrivileges_copy_value);
-      }
+      Map<String,String> __this__addPrivileges = new HashMap<String,String>(other.addPrivileges);
       this.addPrivileges = __this__addPrivileges;
     }
     if (other.isSetDelPrivileges()) {
-      Map<String,String> __this__delPrivileges = new HashMap<String,String>();
-      for (Map.Entry<String, String> other_element : other.delPrivileges.entrySet()) {
-
-        String other_element_key = other_element.getKey();
-        String other_element_value = other_element.getValue();
-
-        String __this__delPrivileges_copy_key = other_element_key;
-
-        String __this__delPrivileges_copy_value = other_element_value;
-
-        __this__delPrivileges.put(__this__delPrivileges_copy_key, __this__delPrivileges_copy_value);
-      }
+      Map<String,String> __this__delPrivileges = new HashMap<String,String>(other.delPrivileges);
       this.delPrivileges = __this__delPrivileges;
     }
   }
@@ -391,60 +373,60 @@ public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChan
 
   @Override
   public int hashCode() {
-    HashCodeBuilder builder = new HashCodeBuilder();
+    List<Object> list = new ArrayList<Object>();
 
     boolean present_authzObj = true && (isSetAuthzObj());
-    builder.append(present_authzObj);
+    list.add(present_authzObj);
     if (present_authzObj)
-      builder.append(authzObj);
+      list.add(authzObj);
 
     boolean present_addPrivileges = true && (isSetAddPrivileges());
-    builder.append(present_addPrivileges);
+    list.add(present_addPrivileges);
     if (present_addPrivileges)
-      builder.append(addPrivileges);
+      list.add(addPrivileges);
 
     boolean present_delPrivileges = true && (isSetDelPrivileges());
-    builder.append(present_delPrivileges);
+    list.add(present_delPrivileges);
     if (present_delPrivileges)
-      builder.append(delPrivileges);
+      list.add(delPrivileges);
 
-    return builder.toHashCode();
+    return list.hashCode();
   }
 
+  @Override
   public int compareTo(TPrivilegeChanges other) {
     if (!getClass().equals(other.getClass())) {
       return getClass().getName().compareTo(other.getClass().getName());
     }
 
     int lastComparison = 0;
-    TPrivilegeChanges typedOther = (TPrivilegeChanges)other;
 
-    lastComparison = Boolean.valueOf(isSetAuthzObj()).compareTo(typedOther.isSetAuthzObj());
+    lastComparison = Boolean.valueOf(isSetAuthzObj()).compareTo(other.isSetAuthzObj());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetAuthzObj()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.authzObj, typedOther.authzObj);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.authzObj, other.authzObj);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetAddPrivileges()).compareTo(typedOther.isSetAddPrivileges());
+    lastComparison = Boolean.valueOf(isSetAddPrivileges()).compareTo(other.isSetAddPrivileges());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetAddPrivileges()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.addPrivileges, typedOther.addPrivileges);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.addPrivileges, other.addPrivileges);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetDelPrivileges()).compareTo(typedOther.isSetDelPrivileges());
+    lastComparison = Boolean.valueOf(isSetDelPrivileges()).compareTo(other.isSetDelPrivileges());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetDelPrivileges()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.delPrivileges, typedOther.delPrivileges);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.delPrivileges, other.delPrivileges);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -560,13 +542,13 @@ public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChan
               {
                 org.apache.thrift.protocol.TMap _map66 = iprot.readMapBegin();
                 struct.addPrivileges = new HashMap<String,String>(2*_map66.size);
-                for (int _i67 = 0; _i67 < _map66.size; ++_i67)
+                String _key67;
+                String _val68;
+                for (int _i69 = 0; _i69 < _map66.size; ++_i69)
                 {
-                  String _key68; // required
-                  String _val69; // required
-                  _key68 = iprot.readString();
-                  _val69 = iprot.readString();
-                  struct.addPrivileges.put(_key68, _val69);
+                  _key67 = iprot.readString();
+                  _val68 = iprot.readString();
+                  struct.addPrivileges.put(_key67, _val68);
                 }
                 iprot.readMapEnd();
               }
@@ -580,13 +562,13 @@ public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChan
               {
                 org.apache.thrift.protocol.TMap _map70 = iprot.readMapBegin();
                 struct.delPrivileges = new HashMap<String,String>(2*_map70.size);
-                for (int _i71 = 0; _i71 < _map70.size; ++_i71)
+                String _key71;
+                String _val72;
+                for (int _i73 = 0; _i73 < _map70.size; ++_i73)
                 {
-                  String _key72; // required
-                  String _val73; // required
-                  _key72 = iprot.readString();
-                  _val73 = iprot.readString();
-                  struct.delPrivileges.put(_key72, _val73);
+                  _key71 = iprot.readString();
+                  _val72 = iprot.readString();
+                  struct.delPrivileges.put(_key71, _val72);
                 }
                 iprot.readMapEnd();
               }
@@ -683,26 +665,26 @@ public class TPrivilegeChanges implements org.apache.thrift.TBase<TPrivilegeChan
       {
         org.apache.thrift.protocol.TMap _map78 = new org.apache.thrift.protocol.TMap(org.apache.thrift.protocol.TType.STRING, org.apache.thrift.protocol.TType.STRING, iprot.readI32());
         struct.addPrivileges = new HashMap<String,String>(2*_map78.size);
-        for (int _i79 = 0; _i79 < _map78.size; ++_i79)
+        String _key79;
+        String _val80;
+        for (int _i81 = 0; _i81 < _map78.size; ++_i81)
         {
-          String _key80; // required
-          String _val81; // required
-          _key80 = iprot.readString();
-          _val81 = iprot.readString();
-          struct.addPrivileges.put(_key80, _val81);
+          _key79 = iprot.readString();
+          _val80 = iprot.readString();
+          struct.addPrivileges.put(_key79, _val80);
         }
       }
       struct.setAddPrivilegesIsSet(true);
       {
         org.apache.thrift.protocol.TMap _map82 = new org.apache.thrift.protocol.TMap(org.apache.thrift.protocol.TType.STRING, org.apache.thrift.protocol.TType.STRING, iprot.readI32());
         struct.delPrivileges = new HashMap<String,String>(2*_map82.size);
-        for (int _i83 = 0; _i83 < _map82.size; ++_i83)
+        String _key83;
+        String _val84;
+        for (int _i85 = 0; _i85 < _map82.size; ++_i85)
         {
-          String _key84; // required
-          String _val85; // required
-          _key84 = iprot.readString();
-          _val85 = iprot.readString();
-          struct.delPrivileges.put(_key84, _val85);
+          _key83 = iprot.readString();
+          _val84 = iprot.readString();
+          struct.delPrivileges.put(_key83, _val84);
         }
       }
       struct.setDelPrivilegesIsSet(true);

http://git-wip-us.apache.org/repos/asf/sentry/blob/e67bde0d/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TRoleChanges.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TRoleChanges.java b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TRoleChanges.java
index 7920872..879f13e 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TRoleChanges.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TRoleChanges.java
@@ -1,12 +1,11 @@
 /**
- * Autogenerated by Thrift Compiler (0.9.0)
+ * Autogenerated by Thrift Compiler (0.9.3)
  *
  * DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
  *  @generated
  */
 package org.apache.sentry.hdfs.service.thrift;
 
-import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.thrift.scheme.IScheme;
 import org.apache.thrift.scheme.SchemeFactory;
 import org.apache.thrift.scheme.StandardScheme;
@@ -16,6 +15,8 @@ import org.apache.thrift.protocol.TTupleProtocol;
 import org.apache.thrift.protocol.TProtocolException;
 import org.apache.thrift.EncodingUtils;
 import org.apache.thrift.TException;
+import org.apache.thrift.async.AsyncMethodCallback;
+import org.apache.thrift.server.AbstractNonblockingServer.*;
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Map;
@@ -28,10 +29,13 @@ import java.util.Collections;
 import java.util.BitSet;
 import java.nio.ByteBuffer;
 import java.util.Arrays;
+import javax.annotation.Generated;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRoleChanges._Fields>, java.io.Serializable, Cloneable {
+@SuppressWarnings({"cast", "rawtypes", "serial", "unchecked"})
+@Generated(value = "Autogenerated by Thrift Compiler (0.9.3)", date = "2016-05-05")
+public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRoleChanges._Fields>, java.io.Serializable, Cloneable, Comparable<TRoleChanges> {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TRoleChanges");
 
   private static final org.apache.thrift.protocol.TField ROLE_FIELD_DESC = new org.apache.thrift.protocol.TField("role", org.apache.thrift.protocol.TType.STRING, (short)1);
@@ -150,17 +154,11 @@ public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRole
       this.role = other.role;
     }
     if (other.isSetAddGroups()) {
-      List<String> __this__addGroups = new ArrayList<String>();
-      for (String other_element : other.addGroups) {
-        __this__addGroups.add(other_element);
-      }
+      List<String> __this__addGroups = new ArrayList<String>(other.addGroups);
       this.addGroups = __this__addGroups;
     }
     if (other.isSetDelGroups()) {
-      List<String> __this__delGroups = new ArrayList<String>();
-      for (String other_element : other.delGroups) {
-        __this__delGroups.add(other_element);
-      }
+      List<String> __this__delGroups = new ArrayList<String>(other.delGroups);
       this.delGroups = __this__delGroups;
     }
   }
@@ -381,60 +379,60 @@ public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRole
 
   @Override
   public int hashCode() {
-    HashCodeBuilder builder = new HashCodeBuilder();
+    List<Object> list = new ArrayList<Object>();
 
     boolean present_role = true && (isSetRole());
-    builder.append(present_role);
+    list.add(present_role);
     if (present_role)
-      builder.append(role);
+      list.add(role);
 
     boolean present_addGroups = true && (isSetAddGroups());
-    builder.append(present_addGroups);
+    list.add(present_addGroups);
     if (present_addGroups)
-      builder.append(addGroups);
+      list.add(addGroups);
 
     boolean present_delGroups = true && (isSetDelGroups());
-    builder.append(present_delGroups);
+    list.add(present_delGroups);
     if (present_delGroups)
-      builder.append(delGroups);
+      list.add(delGroups);
 
-    return builder.toHashCode();
+    return list.hashCode();
   }
 
+  @Override
   public int compareTo(TRoleChanges other) {
     if (!getClass().equals(other.getClass())) {
       return getClass().getName().compareTo(other.getClass().getName());
     }
 
     int lastComparison = 0;
-    TRoleChanges typedOther = (TRoleChanges)other;
 
-    lastComparison = Boolean.valueOf(isSetRole()).compareTo(typedOther.isSetRole());
+    lastComparison = Boolean.valueOf(isSetRole()).compareTo(other.isSetRole());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetRole()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.role, typedOther.role);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.role, other.role);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetAddGroups()).compareTo(typedOther.isSetAddGroups());
+    lastComparison = Boolean.valueOf(isSetAddGroups()).compareTo(other.isSetAddGroups());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetAddGroups()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.addGroups, typedOther.addGroups);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.addGroups, other.addGroups);
       if (lastComparison != 0) {
         return lastComparison;
       }
     }
-    lastComparison = Boolean.valueOf(isSetDelGroups()).compareTo(typedOther.isSetDelGroups());
+    lastComparison = Boolean.valueOf(isSetDelGroups()).compareTo(other.isSetDelGroups());
     if (lastComparison != 0) {
       return lastComparison;
     }
     if (isSetDelGroups()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.delGroups, typedOther.delGroups);
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.delGroups, other.delGroups);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -550,11 +548,11 @@ public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRole
               {
                 org.apache.thrift.protocol.TList _list86 = iprot.readListBegin();
                 struct.addGroups = new ArrayList<String>(_list86.size);
-                for (int _i87 = 0; _i87 < _list86.size; ++_i87)
+                String _elem87;
+                for (int _i88 = 0; _i88 < _list86.size; ++_i88)
                 {
-                  String _elem88; // required
-                  _elem88 = iprot.readString();
-                  struct.addGroups.add(_elem88);
+                  _elem87 = iprot.readString();
+                  struct.addGroups.add(_elem87);
                 }
                 iprot.readListEnd();
               }
@@ -568,11 +566,11 @@ public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRole
               {
                 org.apache.thrift.protocol.TList _list89 = iprot.readListBegin();
                 struct.delGroups = new ArrayList<String>(_list89.size);
-                for (int _i90 = 0; _i90 < _list89.size; ++_i90)
+                String _elem90;
+                for (int _i91 = 0; _i91 < _list89.size; ++_i91)
                 {
-                  String _elem91; // required
-                  _elem91 = iprot.readString();
-                  struct.delGroups.add(_elem91);
+                  _elem90 = iprot.readString();
+                  struct.delGroups.add(_elem90);
                 }
                 iprot.readListEnd();
               }
@@ -665,22 +663,22 @@ public class TRoleChanges implements org.apache.thrift.TBase<TRoleChanges, TRole
       {
         org.apache.thrift.protocol.TList _list96 = new org.apache.thrift.protocol.TList(org.apache.thrift.protocol.TType.STRING, iprot.readI32());
         struct.addGroups = new ArrayList<String>(_list96.size);
-        for (int _i97 = 0; _i97 < _list96.size; ++_i97)
+        String _elem97;
+        for (int _i98 = 0; _i98 < _list96.size; ++_i98)
         {
-          String _elem98; // required
-          _elem98 = iprot.readString();
-          struct.addGroups.add(_elem98);
+          _elem97 = iprot.readString();
+          struct.addGroups.add(_elem97);
         }
       }
       struct.setAddGroupsIsSet(true);
       {
         org.apache.thrift.protocol.TList _list99 = new org.apache.thrift.protocol.TList(org.apache.thrift.protocol.TType.STRING, iprot.readI32());
         struct.delGroups = new ArrayList<String>(_list99.size);
-        for (int _i100 = 0; _i100 < _list99.size; ++_i100)
+        String _elem100;
+        for (int _i101 = 0; _i101 < _list99.size; ++_i101)
         {
-          String _elem101; // required
-          _elem101 = iprot.readString();
-          struct.delGroups.add(_elem101);
+          _elem100 = iprot.readString();
+          struct.delGroups.add(_elem100);
         }
       }
       struct.setDelGroupsIsSet(true);