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

[3/3] git commit: SENTRY-81: Thrift API status should be an object with status code (Brock Noland via Shreepadma Venugopalan)

SENTRY-81: Thrift API status should be an object with status code (Brock Noland via Shreepadma Venugopalan)


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/2a1ceb57
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/2a1ceb57
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/2a1ceb57

Branch: refs/heads/db_policy_store
Commit: 2a1ceb577a044768db63b4d59ff66273e65823fe
Parents: c6bec16
Author: Shreepadma Venugopalan <sh...@apache.org>
Authored: Thu Jan 2 13:02:02 2014 -0800
Committer: Shreepadma Venugopalan <sh...@apache.org>
Committed: Thu Jan 2 13:02:02 2014 -0800

----------------------------------------------------------------------
 sentry-provider/sentry-provider-db/pom.xml      |   1 -
 .../api/SentryThriftPolicyService.java          | 655 ++-----------------
 .../api/TAlterSentryRoleAddGroupsResponse.java  | 133 ++--
 .../TAlterSentryRoleDeleteGroupsResponse.java   | 133 ++--
 .../api/TCreateSentryPrivilegeResponse.java     | 133 ++--
 .../api/TCreateSentryRoleResponse.java          | 133 ++--
 .../api/TListSentryRolesResponse.java           | 133 ++--
 .../api/TSentryAlreadyExistsException.java      | 391 -----------
 .../api/TSentryNoSuchObjectException.java       | 391 -----------
 .../policystore/api/TSentryResponseStatus.java  | 608 +++++++++++++++++
 .../sentry/policystore/api/TSentryStatus.java   |  51 ++
 .../service/persistent/SentryPolicyStore.java   |  43 +-
 .../HiveMetaStoreSentryPolicyStoreHandler.java  |  21 +-
 .../thrift/SentryPolicyStoreHandler.java        |  19 +-
 .../provider/db/service/thrift/Status.java      |  57 ++
 .../db/service/thrift/sentry_policystore.thrift |  48 +-
 16 files changed, 1171 insertions(+), 1779 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/2a1ceb57/sentry-provider/sentry-provider-db/pom.xml
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/pom.xml b/sentry-provider/sentry-provider-db/pom.xml
index 38624c1..ce69e68 100644
--- a/sentry-provider/sentry-provider-db/pom.xml
+++ b/sentry-provider/sentry-provider-db/pom.xml
@@ -103,7 +103,6 @@ limitations under the License.
             </goals>
             <configuration>
               <sources>
-                <source>src/main/java/org/apache/sentry/provider/db/service/model</source>
                 <source>src/gen/thrift/gen-javabean</source>
               </sources>
             </configuration>

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/2a1ceb57/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/SentryThriftPolicyService.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/SentryThriftPolicyService.java b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/SentryThriftPolicyService.java
index c3af9d9..00862f4 100644
--- a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/SentryThriftPolicyService.java
+++ b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/SentryThriftPolicyService.java
@@ -35,15 +35,15 @@ public class SentryThriftPolicyService {
 
   public interface Iface {
 
-    public TCreateSentryRoleResponse create_sentry_role(TCreateSentryRoleRequest request) throws TSentryAlreadyExistsException, org.apache.thrift.TException;
+    public TCreateSentryRoleResponse create_sentry_role(TCreateSentryRoleRequest request) throws org.apache.thrift.TException;
 
-    public TCreateSentryPrivilegeResponse create_sentry_privilege(TCreateSentryPrivilegeRequest request) throws TSentryAlreadyExistsException, org.apache.thrift.TException;
+    public TCreateSentryPrivilegeResponse create_sentry_privilege(TCreateSentryPrivilegeRequest request) throws org.apache.thrift.TException;
 
-    public TAlterSentryRoleAddGroupsResponse alter_sentry_role_add_groups(TAlterSentryRoleAddGroupsRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException;
+    public TAlterSentryRoleAddGroupsResponse alter_sentry_role_add_groups(TAlterSentryRoleAddGroupsRequest request) throws org.apache.thrift.TException;
 
-    public TAlterSentryRoleDeleteGroupsResponse alter_sentry_role_delete_groups(TAlterSentryRoleDeleteGroupsRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException;
+    public TAlterSentryRoleDeleteGroupsResponse alter_sentry_role_delete_groups(TAlterSentryRoleDeleteGroupsRequest request) throws org.apache.thrift.TException;
 
-    public TListSentryRolesResponse list_sentry_roles(TListSentryRolesRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException;
+    public TListSentryRolesResponse list_sentry_roles(TListSentryRolesRequest request) throws org.apache.thrift.TException;
 
   }
 
@@ -81,7 +81,7 @@ public class SentryThriftPolicyService {
       super(iprot, oprot);
     }
 
-    public TCreateSentryRoleResponse create_sentry_role(TCreateSentryRoleRequest request) throws TSentryAlreadyExistsException, org.apache.thrift.TException
+    public TCreateSentryRoleResponse create_sentry_role(TCreateSentryRoleRequest request) throws org.apache.thrift.TException
     {
       send_create_sentry_role(request);
       return recv_create_sentry_role();
@@ -94,20 +94,17 @@ public class SentryThriftPolicyService {
       sendBase("create_sentry_role", args);
     }
 
-    public TCreateSentryRoleResponse recv_create_sentry_role() throws TSentryAlreadyExistsException, org.apache.thrift.TException
+    public TCreateSentryRoleResponse recv_create_sentry_role() throws org.apache.thrift.TException
     {
       create_sentry_role_result result = new create_sentry_role_result();
       receiveBase(result, "create_sentry_role");
       if (result.isSetSuccess()) {
         return result.success;
       }
-      if (result.o1 != null) {
-        throw result.o1;
-      }
       throw new org.apache.thrift.TApplicationException(org.apache.thrift.TApplicationException.MISSING_RESULT, "create_sentry_role failed: unknown result");
     }
 
-    public TCreateSentryPrivilegeResponse create_sentry_privilege(TCreateSentryPrivilegeRequest request) throws TSentryAlreadyExistsException, org.apache.thrift.TException
+    public TCreateSentryPrivilegeResponse create_sentry_privilege(TCreateSentryPrivilegeRequest request) throws org.apache.thrift.TException
     {
       send_create_sentry_privilege(request);
       return recv_create_sentry_privilege();
@@ -120,20 +117,17 @@ public class SentryThriftPolicyService {
       sendBase("create_sentry_privilege", args);
     }
 
-    public TCreateSentryPrivilegeResponse recv_create_sentry_privilege() throws TSentryAlreadyExistsException, org.apache.thrift.TException
+    public TCreateSentryPrivilegeResponse recv_create_sentry_privilege() throws org.apache.thrift.TException
     {
       create_sentry_privilege_result result = new create_sentry_privilege_result();
       receiveBase(result, "create_sentry_privilege");
       if (result.isSetSuccess()) {
         return result.success;
       }
-      if (result.o1 != null) {
-        throw result.o1;
-      }
       throw new org.apache.thrift.TApplicationException(org.apache.thrift.TApplicationException.MISSING_RESULT, "create_sentry_privilege failed: unknown result");
     }
 
-    public TAlterSentryRoleAddGroupsResponse alter_sentry_role_add_groups(TAlterSentryRoleAddGroupsRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TAlterSentryRoleAddGroupsResponse alter_sentry_role_add_groups(TAlterSentryRoleAddGroupsRequest request) throws org.apache.thrift.TException
     {
       send_alter_sentry_role_add_groups(request);
       return recv_alter_sentry_role_add_groups();
@@ -146,20 +140,17 @@ public class SentryThriftPolicyService {
       sendBase("alter_sentry_role_add_groups", args);
     }
 
-    public TAlterSentryRoleAddGroupsResponse recv_alter_sentry_role_add_groups() throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TAlterSentryRoleAddGroupsResponse recv_alter_sentry_role_add_groups() throws org.apache.thrift.TException
     {
       alter_sentry_role_add_groups_result result = new alter_sentry_role_add_groups_result();
       receiveBase(result, "alter_sentry_role_add_groups");
       if (result.isSetSuccess()) {
         return result.success;
       }
-      if (result.o1 != null) {
-        throw result.o1;
-      }
       throw new org.apache.thrift.TApplicationException(org.apache.thrift.TApplicationException.MISSING_RESULT, "alter_sentry_role_add_groups failed: unknown result");
     }
 
-    public TAlterSentryRoleDeleteGroupsResponse alter_sentry_role_delete_groups(TAlterSentryRoleDeleteGroupsRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TAlterSentryRoleDeleteGroupsResponse alter_sentry_role_delete_groups(TAlterSentryRoleDeleteGroupsRequest request) throws org.apache.thrift.TException
     {
       send_alter_sentry_role_delete_groups(request);
       return recv_alter_sentry_role_delete_groups();
@@ -172,20 +163,17 @@ public class SentryThriftPolicyService {
       sendBase("alter_sentry_role_delete_groups", args);
     }
 
-    public TAlterSentryRoleDeleteGroupsResponse recv_alter_sentry_role_delete_groups() throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TAlterSentryRoleDeleteGroupsResponse recv_alter_sentry_role_delete_groups() throws org.apache.thrift.TException
     {
       alter_sentry_role_delete_groups_result result = new alter_sentry_role_delete_groups_result();
       receiveBase(result, "alter_sentry_role_delete_groups");
       if (result.isSetSuccess()) {
         return result.success;
       }
-      if (result.o1 != null) {
-        throw result.o1;
-      }
       throw new org.apache.thrift.TApplicationException(org.apache.thrift.TApplicationException.MISSING_RESULT, "alter_sentry_role_delete_groups failed: unknown result");
     }
 
-    public TListSentryRolesResponse list_sentry_roles(TListSentryRolesRequest request) throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TListSentryRolesResponse list_sentry_roles(TListSentryRolesRequest request) throws org.apache.thrift.TException
     {
       send_list_sentry_roles(request);
       return recv_list_sentry_roles();
@@ -198,16 +186,13 @@ public class SentryThriftPolicyService {
       sendBase("list_sentry_roles", args);
     }
 
-    public TListSentryRolesResponse recv_list_sentry_roles() throws TSentryNoSuchObjectException, org.apache.thrift.TException
+    public TListSentryRolesResponse recv_list_sentry_roles() throws org.apache.thrift.TException
     {
       list_sentry_roles_result result = new list_sentry_roles_result();
       receiveBase(result, "list_sentry_roles");
       if (result.isSetSuccess()) {
         return result.success;
       }
-      if (result.o1 != null) {
-        throw result.o1;
-      }
       throw new org.apache.thrift.TApplicationException(org.apache.thrift.TApplicationException.MISSING_RESULT, "list_sentry_roles failed: unknown result");
     }
 
@@ -251,7 +236,7 @@ public class SentryThriftPolicyService {
         prot.writeMessageEnd();
       }
 
-      public TCreateSentryRoleResponse getResult() throws TSentryAlreadyExistsException, org.apache.thrift.TException {
+      public TCreateSentryRoleResponse getResult() throws org.apache.thrift.TException {
         if (getState() != org.apache.thrift.async.TAsyncMethodCall.State.RESPONSE_READ) {
           throw new IllegalStateException("Method call not finished!");
         }
@@ -283,7 +268,7 @@ public class SentryThriftPolicyService {
         prot.writeMessageEnd();
       }
 
-      public TCreateSentryPrivilegeResponse getResult() throws TSentryAlreadyExistsException, org.apache.thrift.TException {
+      public TCreateSentryPrivilegeResponse getResult() throws org.apache.thrift.TException {
         if (getState() != org.apache.thrift.async.TAsyncMethodCall.State.RESPONSE_READ) {
           throw new IllegalStateException("Method call not finished!");
         }
@@ -315,7 +300,7 @@ public class SentryThriftPolicyService {
         prot.writeMessageEnd();
       }
 
-      public TAlterSentryRoleAddGroupsResponse getResult() throws TSentryNoSuchObjectException, org.apache.thrift.TException {
+      public TAlterSentryRoleAddGroupsResponse getResult() throws org.apache.thrift.TException {
         if (getState() != org.apache.thrift.async.TAsyncMethodCall.State.RESPONSE_READ) {
           throw new IllegalStateException("Method call not finished!");
         }
@@ -347,7 +332,7 @@ public class SentryThriftPolicyService {
         prot.writeMessageEnd();
       }
 
-      public TAlterSentryRoleDeleteGroupsResponse getResult() throws TSentryNoSuchObjectException, org.apache.thrift.TException {
+      public TAlterSentryRoleDeleteGroupsResponse getResult() throws org.apache.thrift.TException {
         if (getState() != org.apache.thrift.async.TAsyncMethodCall.State.RESPONSE_READ) {
           throw new IllegalStateException("Method call not finished!");
         }
@@ -379,7 +364,7 @@ public class SentryThriftPolicyService {
         prot.writeMessageEnd();
       }
 
-      public TListSentryRolesResponse getResult() throws TSentryNoSuchObjectException, org.apache.thrift.TException {
+      public TListSentryRolesResponse getResult() throws org.apache.thrift.TException {
         if (getState() != org.apache.thrift.async.TAsyncMethodCall.State.RESPONSE_READ) {
           throw new IllegalStateException("Method call not finished!");
         }
@@ -425,11 +410,7 @@ public class SentryThriftPolicyService {
 
       public create_sentry_role_result getResult(I iface, create_sentry_role_args args) throws org.apache.thrift.TException {
         create_sentry_role_result result = new create_sentry_role_result();
-        try {
-          result.success = iface.create_sentry_role(args.request);
-        } catch (TSentryAlreadyExistsException o1) {
-          result.o1 = o1;
-        }
+        result.success = iface.create_sentry_role(args.request);
         return result;
       }
     }
@@ -449,11 +430,7 @@ public class SentryThriftPolicyService {
 
       public create_sentry_privilege_result getResult(I iface, create_sentry_privilege_args args) throws org.apache.thrift.TException {
         create_sentry_privilege_result result = new create_sentry_privilege_result();
-        try {
-          result.success = iface.create_sentry_privilege(args.request);
-        } catch (TSentryAlreadyExistsException o1) {
-          result.o1 = o1;
-        }
+        result.success = iface.create_sentry_privilege(args.request);
         return result;
       }
     }
@@ -473,11 +450,7 @@ public class SentryThriftPolicyService {
 
       public alter_sentry_role_add_groups_result getResult(I iface, alter_sentry_role_add_groups_args args) throws org.apache.thrift.TException {
         alter_sentry_role_add_groups_result result = new alter_sentry_role_add_groups_result();
-        try {
-          result.success = iface.alter_sentry_role_add_groups(args.request);
-        } catch (TSentryNoSuchObjectException o1) {
-          result.o1 = o1;
-        }
+        result.success = iface.alter_sentry_role_add_groups(args.request);
         return result;
       }
     }
@@ -497,11 +470,7 @@ public class SentryThriftPolicyService {
 
       public alter_sentry_role_delete_groups_result getResult(I iface, alter_sentry_role_delete_groups_args args) throws org.apache.thrift.TException {
         alter_sentry_role_delete_groups_result result = new alter_sentry_role_delete_groups_result();
-        try {
-          result.success = iface.alter_sentry_role_delete_groups(args.request);
-        } catch (TSentryNoSuchObjectException o1) {
-          result.o1 = o1;
-        }
+        result.success = iface.alter_sentry_role_delete_groups(args.request);
         return result;
       }
     }
@@ -521,11 +490,7 @@ public class SentryThriftPolicyService {
 
       public list_sentry_roles_result getResult(I iface, list_sentry_roles_args args) throws org.apache.thrift.TException {
         list_sentry_roles_result result = new list_sentry_roles_result();
-        try {
-          result.success = iface.list_sentry_roles(args.request);
-        } catch (TSentryNoSuchObjectException o1) {
-          result.o1 = o1;
-        }
+        result.success = iface.list_sentry_roles(args.request);
         return result;
       }
     }
@@ -899,7 +864,6 @@ public class SentryThriftPolicyService {
     private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("create_sentry_role_result");
 
     private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.STRUCT, (short)0);
-    private static final org.apache.thrift.protocol.TField O1_FIELD_DESC = new org.apache.thrift.protocol.TField("o1", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
     private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
     static {
@@ -908,12 +872,10 @@ public class SentryThriftPolicyService {
     }
 
     private TCreateSentryRoleResponse success; // required
-    private TSentryAlreadyExistsException o1; // 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 {
-      SUCCESS((short)0, "success"),
-      O1((short)1, "o1");
+      SUCCESS((short)0, "success");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -930,8 +892,6 @@ public class SentryThriftPolicyService {
         switch(fieldId) {
           case 0: // SUCCESS
             return SUCCESS;
-          case 1: // O1
-            return O1;
           default:
             return null;
         }
@@ -977,8 +937,6 @@ public class SentryThriftPolicyService {
       Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
       tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.DEFAULT, 
           new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TCreateSentryRoleResponse.class)));
-      tmpMap.put(_Fields.O1, new org.apache.thrift.meta_data.FieldMetaData("o1", org.apache.thrift.TFieldRequirementType.DEFAULT, 
-          new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.STRUCT)));
       metaDataMap = Collections.unmodifiableMap(tmpMap);
       org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(create_sentry_role_result.class, metaDataMap);
     }
@@ -987,12 +945,10 @@ public class SentryThriftPolicyService {
     }
 
     public create_sentry_role_result(
-      TCreateSentryRoleResponse success,
-      TSentryAlreadyExistsException o1)
+      TCreateSentryRoleResponse success)
     {
       this();
       this.success = success;
-      this.o1 = o1;
     }
 
     /**
@@ -1002,9 +958,6 @@ public class SentryThriftPolicyService {
       if (other.isSetSuccess()) {
         this.success = new TCreateSentryRoleResponse(other.success);
       }
-      if (other.isSetO1()) {
-        this.o1 = new TSentryAlreadyExistsException(other.o1);
-      }
     }
 
     public create_sentry_role_result deepCopy() {
@@ -1014,7 +967,6 @@ public class SentryThriftPolicyService {
     @Override
     public void clear() {
       this.success = null;
-      this.o1 = null;
     }
 
     public TCreateSentryRoleResponse getSuccess() {
@@ -1040,29 +992,6 @@ public class SentryThriftPolicyService {
       }
     }
 
-    public TSentryAlreadyExistsException getO1() {
-      return this.o1;
-    }
-
-    public void setO1(TSentryAlreadyExistsException o1) {
-      this.o1 = o1;
-    }
-
-    public void unsetO1() {
-      this.o1 = null;
-    }
-
-    /** Returns true if field o1 is set (has been assigned a value) and false otherwise */
-    public boolean isSetO1() {
-      return this.o1 != null;
-    }
-
-    public void setO1IsSet(boolean value) {
-      if (!value) {
-        this.o1 = null;
-      }
-    }
-
     public void setFieldValue(_Fields field, Object value) {
       switch (field) {
       case SUCCESS:
@@ -1073,14 +1002,6 @@ public class SentryThriftPolicyService {
         }
         break;
 
-      case O1:
-        if (value == null) {
-          unsetO1();
-        } else {
-          setO1((TSentryAlreadyExistsException)value);
-        }
-        break;
-
       }
     }
 
@@ -1089,9 +1010,6 @@ public class SentryThriftPolicyService {
       case SUCCESS:
         return getSuccess();
 
-      case O1:
-        return getO1();
-
       }
       throw new IllegalStateException();
     }
@@ -1105,8 +1023,6 @@ public class SentryThriftPolicyService {
       switch (field) {
       case SUCCESS:
         return isSetSuccess();
-      case O1:
-        return isSetO1();
       }
       throw new IllegalStateException();
     }
@@ -1133,15 +1049,6 @@ public class SentryThriftPolicyService {
           return false;
       }
 
-      boolean this_present_o1 = true && this.isSetO1();
-      boolean that_present_o1 = true && that.isSetO1();
-      if (this_present_o1 || that_present_o1) {
-        if (!(this_present_o1 && that_present_o1))
-          return false;
-        if (!this.o1.equals(that.o1))
-          return false;
-      }
-
       return true;
     }
 
@@ -1154,11 +1061,6 @@ public class SentryThriftPolicyService {
       if (present_success)
         builder.append(success);
 
-      boolean present_o1 = true && (isSetO1());
-      builder.append(present_o1);
-      if (present_o1)
-        builder.append(o1);
-
       return builder.toHashCode();
     }
 
@@ -1180,16 +1082,6 @@ public class SentryThriftPolicyService {
           return lastComparison;
         }
       }
-      lastComparison = Boolean.valueOf(isSetO1()).compareTo(typedOther.isSetO1());
-      if (lastComparison != 0) {
-        return lastComparison;
-      }
-      if (isSetO1()) {
-        lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.o1, typedOther.o1);
-        if (lastComparison != 0) {
-          return lastComparison;
-        }
-      }
       return 0;
     }
 
@@ -1217,14 +1109,6 @@ public class SentryThriftPolicyService {
         sb.append(this.success);
       }
       first = false;
-      if (!first) sb.append(", ");
-      sb.append("o1:");
-      if (this.o1 == null) {
-        sb.append("null");
-      } else {
-        sb.append(this.o1);
-      }
-      first = false;
       sb.append(")");
       return sb.toString();
     }
@@ -1280,15 +1164,6 @@ public class SentryThriftPolicyService {
                 org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
               }
               break;
-            case 1: // O1
-              if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
-                struct.o1 = new TSentryAlreadyExistsException();
-                struct.o1.read(iprot);
-                struct.setO1IsSet(true);
-              } else { 
-                org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
-              }
-              break;
             default:
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
           }
@@ -1307,11 +1182,6 @@ public class SentryThriftPolicyService {
           struct.success.write(oprot);
           oprot.writeFieldEnd();
         }
-        if (struct.o1 != null) {
-          oprot.writeFieldBegin(O1_FIELD_DESC);
-          struct.o1.write(oprot);
-          oprot.writeFieldEnd();
-        }
         oprot.writeFieldStop();
         oprot.writeStructEnd();
       }
@@ -1333,32 +1203,21 @@ public class SentryThriftPolicyService {
         if (struct.isSetSuccess()) {
           optionals.set(0);
         }
-        if (struct.isSetO1()) {
-          optionals.set(1);
-        }
-        oprot.writeBitSet(optionals, 2);
+        oprot.writeBitSet(optionals, 1);
         if (struct.isSetSuccess()) {
           struct.success.write(oprot);
         }
-        if (struct.isSetO1()) {
-          struct.o1.write(oprot);
-        }
       }
 
       @Override
       public void read(org.apache.thrift.protocol.TProtocol prot, create_sentry_role_result struct) throws org.apache.thrift.TException {
         TTupleProtocol iprot = (TTupleProtocol) prot;
-        BitSet incoming = iprot.readBitSet(2);
+        BitSet incoming = iprot.readBitSet(1);
         if (incoming.get(0)) {
           struct.success = new TCreateSentryRoleResponse();
           struct.success.read(iprot);
           struct.setSuccessIsSet(true);
         }
-        if (incoming.get(1)) {
-          struct.o1 = new TSentryAlreadyExistsException();
-          struct.o1.read(iprot);
-          struct.setO1IsSet(true);
-        }
       }
     }
 
@@ -1731,7 +1590,6 @@ public class SentryThriftPolicyService {
     private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("create_sentry_privilege_result");
 
     private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.STRUCT, (short)0);
-    private static final org.apache.thrift.protocol.TField O1_FIELD_DESC = new org.apache.thrift.protocol.TField("o1", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
     private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
     static {
@@ -1740,12 +1598,10 @@ public class SentryThriftPolicyService {
     }
 
     private TCreateSentryPrivilegeResponse success; // required
-    private TSentryAlreadyExistsException o1; // 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 {
-      SUCCESS((short)0, "success"),
-      O1((short)1, "o1");
+      SUCCESS((short)0, "success");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -1762,8 +1618,6 @@ public class SentryThriftPolicyService {
         switch(fieldId) {
           case 0: // SUCCESS
             return SUCCESS;
-          case 1: // O1
-            return O1;
           default:
             return null;
         }
@@ -1809,8 +1663,6 @@ public class SentryThriftPolicyService {
       Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
       tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.DEFAULT, 
           new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TCreateSentryPrivilegeResponse.class)));
-      tmpMap.put(_Fields.O1, new org.apache.thrift.meta_data.FieldMetaData("o1", org.apache.thrift.TFieldRequirementType.DEFAULT, 
-          new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.STRUCT)));
       metaDataMap = Collections.unmodifiableMap(tmpMap);
       org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(create_sentry_privilege_result.class, metaDataMap);
     }
@@ -1819,12 +1671,10 @@ public class SentryThriftPolicyService {
     }
 
     public create_sentry_privilege_result(
-      TCreateSentryPrivilegeResponse success,
-      TSentryAlreadyExistsException o1)
+      TCreateSentryPrivilegeResponse success)
     {
       this();
       this.success = success;
-      this.o1 = o1;
     }
 
     /**
@@ -1834,9 +1684,6 @@ public class SentryThriftPolicyService {
       if (other.isSetSuccess()) {
         this.success = new TCreateSentryPrivilegeResponse(other.success);
       }
-      if (other.isSetO1()) {
-        this.o1 = new TSentryAlreadyExistsException(other.o1);
-      }
     }
 
     public create_sentry_privilege_result deepCopy() {
@@ -1846,7 +1693,6 @@ public class SentryThriftPolicyService {
     @Override
     public void clear() {
       this.success = null;
-      this.o1 = null;
     }
 
     public TCreateSentryPrivilegeResponse getSuccess() {
@@ -1872,29 +1718,6 @@ public class SentryThriftPolicyService {
       }
     }
 
-    public TSentryAlreadyExistsException getO1() {
-      return this.o1;
-    }
-
-    public void setO1(TSentryAlreadyExistsException o1) {
-      this.o1 = o1;
-    }
-
-    public void unsetO1() {
-      this.o1 = null;
-    }
-
-    /** Returns true if field o1 is set (has been assigned a value) and false otherwise */
-    public boolean isSetO1() {
-      return this.o1 != null;
-    }
-
-    public void setO1IsSet(boolean value) {
-      if (!value) {
-        this.o1 = null;
-      }
-    }
-
     public void setFieldValue(_Fields field, Object value) {
       switch (field) {
       case SUCCESS:
@@ -1905,14 +1728,6 @@ public class SentryThriftPolicyService {
         }
         break;
 
-      case O1:
-        if (value == null) {
-          unsetO1();
-        } else {
-          setO1((TSentryAlreadyExistsException)value);
-        }
-        break;
-
       }
     }
 
@@ -1921,9 +1736,6 @@ public class SentryThriftPolicyService {
       case SUCCESS:
         return getSuccess();
 
-      case O1:
-        return getO1();
-
       }
       throw new IllegalStateException();
     }
@@ -1937,8 +1749,6 @@ public class SentryThriftPolicyService {
       switch (field) {
       case SUCCESS:
         return isSetSuccess();
-      case O1:
-        return isSetO1();
       }
       throw new IllegalStateException();
     }
@@ -1965,15 +1775,6 @@ public class SentryThriftPolicyService {
           return false;
       }
 
-      boolean this_present_o1 = true && this.isSetO1();
-      boolean that_present_o1 = true && that.isSetO1();
-      if (this_present_o1 || that_present_o1) {
-        if (!(this_present_o1 && that_present_o1))
-          return false;
-        if (!this.o1.equals(that.o1))
-          return false;
-      }
-
       return true;
     }
 
@@ -1986,11 +1787,6 @@ public class SentryThriftPolicyService {
       if (present_success)
         builder.append(success);
 
-      boolean present_o1 = true && (isSetO1());
-      builder.append(present_o1);
-      if (present_o1)
-        builder.append(o1);
-
       return builder.toHashCode();
     }
 
@@ -2012,16 +1808,6 @@ public class SentryThriftPolicyService {
           return lastComparison;
         }
       }
-      lastComparison = Boolean.valueOf(isSetO1()).compareTo(typedOther.isSetO1());
-      if (lastComparison != 0) {
-        return lastComparison;
-      }
-      if (isSetO1()) {
-        lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.o1, typedOther.o1);
-        if (lastComparison != 0) {
-          return lastComparison;
-        }
-      }
       return 0;
     }
 
@@ -2049,14 +1835,6 @@ public class SentryThriftPolicyService {
         sb.append(this.success);
       }
       first = false;
-      if (!first) sb.append(", ");
-      sb.append("o1:");
-      if (this.o1 == null) {
-        sb.append("null");
-      } else {
-        sb.append(this.o1);
-      }
-      first = false;
       sb.append(")");
       return sb.toString();
     }
@@ -2112,15 +1890,6 @@ public class SentryThriftPolicyService {
                 org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
               }
               break;
-            case 1: // O1
-              if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
-                struct.o1 = new TSentryAlreadyExistsException();
-                struct.o1.read(iprot);
-                struct.setO1IsSet(true);
-              } else { 
-                org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
-              }
-              break;
             default:
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
           }
@@ -2139,11 +1908,6 @@ public class SentryThriftPolicyService {
           struct.success.write(oprot);
           oprot.writeFieldEnd();
         }
-        if (struct.o1 != null) {
-          oprot.writeFieldBegin(O1_FIELD_DESC);
-          struct.o1.write(oprot);
-          oprot.writeFieldEnd();
-        }
         oprot.writeFieldStop();
         oprot.writeStructEnd();
       }
@@ -2165,32 +1929,21 @@ public class SentryThriftPolicyService {
         if (struct.isSetSuccess()) {
           optionals.set(0);
         }
-        if (struct.isSetO1()) {
-          optionals.set(1);
-        }
-        oprot.writeBitSet(optionals, 2);
+        oprot.writeBitSet(optionals, 1);
         if (struct.isSetSuccess()) {
           struct.success.write(oprot);
         }
-        if (struct.isSetO1()) {
-          struct.o1.write(oprot);
-        }
       }
 
       @Override
       public void read(org.apache.thrift.protocol.TProtocol prot, create_sentry_privilege_result struct) throws org.apache.thrift.TException {
         TTupleProtocol iprot = (TTupleProtocol) prot;
-        BitSet incoming = iprot.readBitSet(2);
+        BitSet incoming = iprot.readBitSet(1);
         if (incoming.get(0)) {
           struct.success = new TCreateSentryPrivilegeResponse();
           struct.success.read(iprot);
           struct.setSuccessIsSet(true);
         }
-        if (incoming.get(1)) {
-          struct.o1 = new TSentryAlreadyExistsException();
-          struct.o1.read(iprot);
-          struct.setO1IsSet(true);
-        }
       }
     }
 
@@ -2563,7 +2316,6 @@ public class SentryThriftPolicyService {
     private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("alter_sentry_role_add_groups_result");
 
     private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.STRUCT, (short)0);
-    private static final org.apache.thrift.protocol.TField O1_FIELD_DESC = new org.apache.thrift.protocol.TField("o1", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
     private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
     static {
@@ -2572,12 +2324,10 @@ public class SentryThriftPolicyService {
     }
 
     private TAlterSentryRoleAddGroupsResponse success; // required
-    private TSentryNoSuchObjectException o1; // 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 {
-      SUCCESS((short)0, "success"),
-      O1((short)1, "o1");
+      SUCCESS((short)0, "success");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -2594,8 +2344,6 @@ public class SentryThriftPolicyService {
         switch(fieldId) {
           case 0: // SUCCESS
             return SUCCESS;
-          case 1: // O1
-            return O1;
           default:
             return null;
         }
@@ -2641,8 +2389,6 @@ public class SentryThriftPolicyService {
       Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
       tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.DEFAULT, 
           new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TAlterSentryRoleAddGroupsResponse.class)));
-      tmpMap.put(_Fields.O1, new org.apache.thrift.meta_data.FieldMetaData("o1", org.apache.thrift.TFieldRequirementType.DEFAULT, 
-          new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.STRUCT)));
       metaDataMap = Collections.unmodifiableMap(tmpMap);
       org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(alter_sentry_role_add_groups_result.class, metaDataMap);
     }
@@ -2651,12 +2397,10 @@ public class SentryThriftPolicyService {
     }
 
     public alter_sentry_role_add_groups_result(
-      TAlterSentryRoleAddGroupsResponse success,
-      TSentryNoSuchObjectException o1)
+      TAlterSentryRoleAddGroupsResponse success)
     {
       this();
       this.success = success;
-      this.o1 = o1;
     }
 
     /**
@@ -2666,9 +2410,6 @@ public class SentryThriftPolicyService {
       if (other.isSetSuccess()) {
         this.success = new TAlterSentryRoleAddGroupsResponse(other.success);
       }
-      if (other.isSetO1()) {
-        this.o1 = new TSentryNoSuchObjectException(other.o1);
-      }
     }
 
     public alter_sentry_role_add_groups_result deepCopy() {
@@ -2678,7 +2419,6 @@ public class SentryThriftPolicyService {
     @Override
     public void clear() {
       this.success = null;
-      this.o1 = null;
     }
 
     public TAlterSentryRoleAddGroupsResponse getSuccess() {
@@ -2704,29 +2444,6 @@ public class SentryThriftPolicyService {
       }
     }
 
-    public TSentryNoSuchObjectException getO1() {
-      return this.o1;
-    }
-
-    public void setO1(TSentryNoSuchObjectException o1) {
-      this.o1 = o1;
-    }
-
-    public void unsetO1() {
-      this.o1 = null;
-    }
-
-    /** Returns true if field o1 is set (has been assigned a value) and false otherwise */
-    public boolean isSetO1() {
-      return this.o1 != null;
-    }
-
-    public void setO1IsSet(boolean value) {
-      if (!value) {
-        this.o1 = null;
-      }
-    }
-
     public void setFieldValue(_Fields field, Object value) {
       switch (field) {
       case SUCCESS:
@@ -2737,14 +2454,6 @@ public class SentryThriftPolicyService {
         }
         break;
 
-      case O1:
-        if (value == null) {
-          unsetO1();
-        } else {
-          setO1((TSentryNoSuchObjectException)value);
-        }
-        break;
-
       }
     }
 
@@ -2753,9 +2462,6 @@ public class SentryThriftPolicyService {
       case SUCCESS:
         return getSuccess();
 
-      case O1:
-        return getO1();
-
       }
       throw new IllegalStateException();
     }
@@ -2769,8 +2475,6 @@ public class SentryThriftPolicyService {
       switch (field) {
       case SUCCESS:
         return isSetSuccess();
-      case O1:
-        return isSetO1();
       }
       throw new IllegalStateException();
     }
@@ -2797,15 +2501,6 @@ public class SentryThriftPolicyService {
           return false;
       }
 
-      boolean this_present_o1 = true && this.isSetO1();
-      boolean that_present_o1 = true && that.isSetO1();
-      if (this_present_o1 || that_present_o1) {
-        if (!(this_present_o1 && that_present_o1))
-          return false;
-        if (!this.o1.equals(that.o1))
-          return false;
-      }
-
       return true;
     }
 
@@ -2818,11 +2513,6 @@ public class SentryThriftPolicyService {
       if (present_success)
         builder.append(success);
 
-      boolean present_o1 = true && (isSetO1());
-      builder.append(present_o1);
-      if (present_o1)
-        builder.append(o1);
-
       return builder.toHashCode();
     }
 
@@ -2844,16 +2534,6 @@ public class SentryThriftPolicyService {
           return lastComparison;
         }
       }
-      lastComparison = Boolean.valueOf(isSetO1()).compareTo(typedOther.isSetO1());
-      if (lastComparison != 0) {
-        return lastComparison;
-      }
-      if (isSetO1()) {
-        lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.o1, typedOther.o1);
-        if (lastComparison != 0) {
-          return lastComparison;
-        }
-      }
       return 0;
     }
 
@@ -2881,14 +2561,6 @@ public class SentryThriftPolicyService {
         sb.append(this.success);
       }
       first = false;
-      if (!first) sb.append(", ");
-      sb.append("o1:");
-      if (this.o1 == null) {
-        sb.append("null");
-      } else {
-        sb.append(this.o1);
-      }
-      first = false;
       sb.append(")");
       return sb.toString();
     }
@@ -2944,15 +2616,6 @@ public class SentryThriftPolicyService {
                 org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
               }
               break;
-            case 1: // O1
-              if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
-                struct.o1 = new TSentryNoSuchObjectException();
-                struct.o1.read(iprot);
-                struct.setO1IsSet(true);
-              } else { 
-                org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
-              }
-              break;
             default:
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
           }
@@ -2971,11 +2634,6 @@ public class SentryThriftPolicyService {
           struct.success.write(oprot);
           oprot.writeFieldEnd();
         }
-        if (struct.o1 != null) {
-          oprot.writeFieldBegin(O1_FIELD_DESC);
-          struct.o1.write(oprot);
-          oprot.writeFieldEnd();
-        }
         oprot.writeFieldStop();
         oprot.writeStructEnd();
       }
@@ -2997,32 +2655,21 @@ public class SentryThriftPolicyService {
         if (struct.isSetSuccess()) {
           optionals.set(0);
         }
-        if (struct.isSetO1()) {
-          optionals.set(1);
-        }
-        oprot.writeBitSet(optionals, 2);
+        oprot.writeBitSet(optionals, 1);
         if (struct.isSetSuccess()) {
           struct.success.write(oprot);
         }
-        if (struct.isSetO1()) {
-          struct.o1.write(oprot);
-        }
       }
 
       @Override
       public void read(org.apache.thrift.protocol.TProtocol prot, alter_sentry_role_add_groups_result struct) throws org.apache.thrift.TException {
         TTupleProtocol iprot = (TTupleProtocol) prot;
-        BitSet incoming = iprot.readBitSet(2);
+        BitSet incoming = iprot.readBitSet(1);
         if (incoming.get(0)) {
           struct.success = new TAlterSentryRoleAddGroupsResponse();
           struct.success.read(iprot);
           struct.setSuccessIsSet(true);
         }
-        if (incoming.get(1)) {
-          struct.o1 = new TSentryNoSuchObjectException();
-          struct.o1.read(iprot);
-          struct.setO1IsSet(true);
-        }
       }
     }
 
@@ -3395,7 +3042,6 @@ public class SentryThriftPolicyService {
     private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("alter_sentry_role_delete_groups_result");
 
     private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.STRUCT, (short)0);
-    private static final org.apache.thrift.protocol.TField O1_FIELD_DESC = new org.apache.thrift.protocol.TField("o1", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
     private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
     static {
@@ -3404,12 +3050,10 @@ public class SentryThriftPolicyService {
     }
 
     private TAlterSentryRoleDeleteGroupsResponse success; // required
-    private TSentryNoSuchObjectException o1; // 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 {
-      SUCCESS((short)0, "success"),
-      O1((short)1, "o1");
+      SUCCESS((short)0, "success");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -3426,8 +3070,6 @@ public class SentryThriftPolicyService {
         switch(fieldId) {
           case 0: // SUCCESS
             return SUCCESS;
-          case 1: // O1
-            return O1;
           default:
             return null;
         }
@@ -3473,8 +3115,6 @@ public class SentryThriftPolicyService {
       Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
       tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.DEFAULT, 
           new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TAlterSentryRoleDeleteGroupsResponse.class)));
-      tmpMap.put(_Fields.O1, new org.apache.thrift.meta_data.FieldMetaData("o1", org.apache.thrift.TFieldRequirementType.DEFAULT, 
-          new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.STRUCT)));
       metaDataMap = Collections.unmodifiableMap(tmpMap);
       org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(alter_sentry_role_delete_groups_result.class, metaDataMap);
     }
@@ -3483,12 +3123,10 @@ public class SentryThriftPolicyService {
     }
 
     public alter_sentry_role_delete_groups_result(
-      TAlterSentryRoleDeleteGroupsResponse success,
-      TSentryNoSuchObjectException o1)
+      TAlterSentryRoleDeleteGroupsResponse success)
     {
       this();
       this.success = success;
-      this.o1 = o1;
     }
 
     /**
@@ -3498,9 +3136,6 @@ public class SentryThriftPolicyService {
       if (other.isSetSuccess()) {
         this.success = new TAlterSentryRoleDeleteGroupsResponse(other.success);
       }
-      if (other.isSetO1()) {
-        this.o1 = new TSentryNoSuchObjectException(other.o1);
-      }
     }
 
     public alter_sentry_role_delete_groups_result deepCopy() {
@@ -3510,7 +3145,6 @@ public class SentryThriftPolicyService {
     @Override
     public void clear() {
       this.success = null;
-      this.o1 = null;
     }
 
     public TAlterSentryRoleDeleteGroupsResponse getSuccess() {
@@ -3536,29 +3170,6 @@ public class SentryThriftPolicyService {
       }
     }
 
-    public TSentryNoSuchObjectException getO1() {
-      return this.o1;
-    }
-
-    public void setO1(TSentryNoSuchObjectException o1) {
-      this.o1 = o1;
-    }
-
-    public void unsetO1() {
-      this.o1 = null;
-    }
-
-    /** Returns true if field o1 is set (has been assigned a value) and false otherwise */
-    public boolean isSetO1() {
-      return this.o1 != null;
-    }
-
-    public void setO1IsSet(boolean value) {
-      if (!value) {
-        this.o1 = null;
-      }
-    }
-
     public void setFieldValue(_Fields field, Object value) {
       switch (field) {
       case SUCCESS:
@@ -3569,14 +3180,6 @@ public class SentryThriftPolicyService {
         }
         break;
 
-      case O1:
-        if (value == null) {
-          unsetO1();
-        } else {
-          setO1((TSentryNoSuchObjectException)value);
-        }
-        break;
-
       }
     }
 
@@ -3585,9 +3188,6 @@ public class SentryThriftPolicyService {
       case SUCCESS:
         return getSuccess();
 
-      case O1:
-        return getO1();
-
       }
       throw new IllegalStateException();
     }
@@ -3601,8 +3201,6 @@ public class SentryThriftPolicyService {
       switch (field) {
       case SUCCESS:
         return isSetSuccess();
-      case O1:
-        return isSetO1();
       }
       throw new IllegalStateException();
     }
@@ -3629,15 +3227,6 @@ public class SentryThriftPolicyService {
           return false;
       }
 
-      boolean this_present_o1 = true && this.isSetO1();
-      boolean that_present_o1 = true && that.isSetO1();
-      if (this_present_o1 || that_present_o1) {
-        if (!(this_present_o1 && that_present_o1))
-          return false;
-        if (!this.o1.equals(that.o1))
-          return false;
-      }
-
       return true;
     }
 
@@ -3650,11 +3239,6 @@ public class SentryThriftPolicyService {
       if (present_success)
         builder.append(success);
 
-      boolean present_o1 = true && (isSetO1());
-      builder.append(present_o1);
-      if (present_o1)
-        builder.append(o1);
-
       return builder.toHashCode();
     }
 
@@ -3676,16 +3260,6 @@ public class SentryThriftPolicyService {
           return lastComparison;
         }
       }
-      lastComparison = Boolean.valueOf(isSetO1()).compareTo(typedOther.isSetO1());
-      if (lastComparison != 0) {
-        return lastComparison;
-      }
-      if (isSetO1()) {
-        lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.o1, typedOther.o1);
-        if (lastComparison != 0) {
-          return lastComparison;
-        }
-      }
       return 0;
     }
 
@@ -3713,14 +3287,6 @@ public class SentryThriftPolicyService {
         sb.append(this.success);
       }
       first = false;
-      if (!first) sb.append(", ");
-      sb.append("o1:");
-      if (this.o1 == null) {
-        sb.append("null");
-      } else {
-        sb.append(this.o1);
-      }
-      first = false;
       sb.append(")");
       return sb.toString();
     }
@@ -3776,15 +3342,6 @@ public class SentryThriftPolicyService {
                 org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
               }
               break;
-            case 1: // O1
-              if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
-                struct.o1 = new TSentryNoSuchObjectException();
-                struct.o1.read(iprot);
-                struct.setO1IsSet(true);
-              } else { 
-                org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
-              }
-              break;
             default:
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
           }
@@ -3803,11 +3360,6 @@ public class SentryThriftPolicyService {
           struct.success.write(oprot);
           oprot.writeFieldEnd();
         }
-        if (struct.o1 != null) {
-          oprot.writeFieldBegin(O1_FIELD_DESC);
-          struct.o1.write(oprot);
-          oprot.writeFieldEnd();
-        }
         oprot.writeFieldStop();
         oprot.writeStructEnd();
       }
@@ -3829,32 +3381,21 @@ public class SentryThriftPolicyService {
         if (struct.isSetSuccess()) {
           optionals.set(0);
         }
-        if (struct.isSetO1()) {
-          optionals.set(1);
-        }
-        oprot.writeBitSet(optionals, 2);
+        oprot.writeBitSet(optionals, 1);
         if (struct.isSetSuccess()) {
           struct.success.write(oprot);
         }
-        if (struct.isSetO1()) {
-          struct.o1.write(oprot);
-        }
       }
 
       @Override
       public void read(org.apache.thrift.protocol.TProtocol prot, alter_sentry_role_delete_groups_result struct) throws org.apache.thrift.TException {
         TTupleProtocol iprot = (TTupleProtocol) prot;
-        BitSet incoming = iprot.readBitSet(2);
+        BitSet incoming = iprot.readBitSet(1);
         if (incoming.get(0)) {
           struct.success = new TAlterSentryRoleDeleteGroupsResponse();
           struct.success.read(iprot);
           struct.setSuccessIsSet(true);
         }
-        if (incoming.get(1)) {
-          struct.o1 = new TSentryNoSuchObjectException();
-          struct.o1.read(iprot);
-          struct.setO1IsSet(true);
-        }
       }
     }
 
@@ -4227,7 +3768,6 @@ public class SentryThriftPolicyService {
     private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("list_sentry_roles_result");
 
     private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.STRUCT, (short)0);
-    private static final org.apache.thrift.protocol.TField O1_FIELD_DESC = new org.apache.thrift.protocol.TField("o1", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
     private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
     static {
@@ -4236,12 +3776,10 @@ public class SentryThriftPolicyService {
     }
 
     private TListSentryRolesResponse success; // required
-    private TSentryNoSuchObjectException o1; // 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 {
-      SUCCESS((short)0, "success"),
-      O1((short)1, "o1");
+      SUCCESS((short)0, "success");
 
       private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -4258,8 +3796,6 @@ public class SentryThriftPolicyService {
         switch(fieldId) {
           case 0: // SUCCESS
             return SUCCESS;
-          case 1: // O1
-            return O1;
           default:
             return null;
         }
@@ -4305,8 +3841,6 @@ public class SentryThriftPolicyService {
       Map<_Fields, org.apache.thrift.meta_data.FieldMetaData> tmpMap = new EnumMap<_Fields, org.apache.thrift.meta_data.FieldMetaData>(_Fields.class);
       tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.DEFAULT, 
           new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TListSentryRolesResponse.class)));
-      tmpMap.put(_Fields.O1, new org.apache.thrift.meta_data.FieldMetaData("o1", org.apache.thrift.TFieldRequirementType.DEFAULT, 
-          new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.STRUCT)));
       metaDataMap = Collections.unmodifiableMap(tmpMap);
       org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(list_sentry_roles_result.class, metaDataMap);
     }
@@ -4315,12 +3849,10 @@ public class SentryThriftPolicyService {
     }
 
     public list_sentry_roles_result(
-      TListSentryRolesResponse success,
-      TSentryNoSuchObjectException o1)
+      TListSentryRolesResponse success)
     {
       this();
       this.success = success;
-      this.o1 = o1;
     }
 
     /**
@@ -4330,9 +3862,6 @@ public class SentryThriftPolicyService {
       if (other.isSetSuccess()) {
         this.success = new TListSentryRolesResponse(other.success);
       }
-      if (other.isSetO1()) {
-        this.o1 = new TSentryNoSuchObjectException(other.o1);
-      }
     }
 
     public list_sentry_roles_result deepCopy() {
@@ -4342,7 +3871,6 @@ public class SentryThriftPolicyService {
     @Override
     public void clear() {
       this.success = null;
-      this.o1 = null;
     }
 
     public TListSentryRolesResponse getSuccess() {
@@ -4368,29 +3896,6 @@ public class SentryThriftPolicyService {
       }
     }
 
-    public TSentryNoSuchObjectException getO1() {
-      return this.o1;
-    }
-
-    public void setO1(TSentryNoSuchObjectException o1) {
-      this.o1 = o1;
-    }
-
-    public void unsetO1() {
-      this.o1 = null;
-    }
-
-    /** Returns true if field o1 is set (has been assigned a value) and false otherwise */
-    public boolean isSetO1() {
-      return this.o1 != null;
-    }
-
-    public void setO1IsSet(boolean value) {
-      if (!value) {
-        this.o1 = null;
-      }
-    }
-
     public void setFieldValue(_Fields field, Object value) {
       switch (field) {
       case SUCCESS:
@@ -4401,14 +3906,6 @@ public class SentryThriftPolicyService {
         }
         break;
 
-      case O1:
-        if (value == null) {
-          unsetO1();
-        } else {
-          setO1((TSentryNoSuchObjectException)value);
-        }
-        break;
-
       }
     }
 
@@ -4417,9 +3914,6 @@ public class SentryThriftPolicyService {
       case SUCCESS:
         return getSuccess();
 
-      case O1:
-        return getO1();
-
       }
       throw new IllegalStateException();
     }
@@ -4433,8 +3927,6 @@ public class SentryThriftPolicyService {
       switch (field) {
       case SUCCESS:
         return isSetSuccess();
-      case O1:
-        return isSetO1();
       }
       throw new IllegalStateException();
     }
@@ -4461,15 +3953,6 @@ public class SentryThriftPolicyService {
           return false;
       }
 
-      boolean this_present_o1 = true && this.isSetO1();
-      boolean that_present_o1 = true && that.isSetO1();
-      if (this_present_o1 || that_present_o1) {
-        if (!(this_present_o1 && that_present_o1))
-          return false;
-        if (!this.o1.equals(that.o1))
-          return false;
-      }
-
       return true;
     }
 
@@ -4482,11 +3965,6 @@ public class SentryThriftPolicyService {
       if (present_success)
         builder.append(success);
 
-      boolean present_o1 = true && (isSetO1());
-      builder.append(present_o1);
-      if (present_o1)
-        builder.append(o1);
-
       return builder.toHashCode();
     }
 
@@ -4508,16 +3986,6 @@ public class SentryThriftPolicyService {
           return lastComparison;
         }
       }
-      lastComparison = Boolean.valueOf(isSetO1()).compareTo(typedOther.isSetO1());
-      if (lastComparison != 0) {
-        return lastComparison;
-      }
-      if (isSetO1()) {
-        lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.o1, typedOther.o1);
-        if (lastComparison != 0) {
-          return lastComparison;
-        }
-      }
       return 0;
     }
 
@@ -4545,14 +4013,6 @@ public class SentryThriftPolicyService {
         sb.append(this.success);
       }
       first = false;
-      if (!first) sb.append(", ");
-      sb.append("o1:");
-      if (this.o1 == null) {
-        sb.append("null");
-      } else {
-        sb.append(this.o1);
-      }
-      first = false;
       sb.append(")");
       return sb.toString();
     }
@@ -4608,15 +4068,6 @@ public class SentryThriftPolicyService {
                 org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
               }
               break;
-            case 1: // O1
-              if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
-                struct.o1 = new TSentryNoSuchObjectException();
-                struct.o1.read(iprot);
-                struct.setO1IsSet(true);
-              } else { 
-                org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
-              }
-              break;
             default:
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
           }
@@ -4635,11 +4086,6 @@ public class SentryThriftPolicyService {
           struct.success.write(oprot);
           oprot.writeFieldEnd();
         }
-        if (struct.o1 != null) {
-          oprot.writeFieldBegin(O1_FIELD_DESC);
-          struct.o1.write(oprot);
-          oprot.writeFieldEnd();
-        }
         oprot.writeFieldStop();
         oprot.writeStructEnd();
       }
@@ -4661,32 +4107,21 @@ public class SentryThriftPolicyService {
         if (struct.isSetSuccess()) {
           optionals.set(0);
         }
-        if (struct.isSetO1()) {
-          optionals.set(1);
-        }
-        oprot.writeBitSet(optionals, 2);
+        oprot.writeBitSet(optionals, 1);
         if (struct.isSetSuccess()) {
           struct.success.write(oprot);
         }
-        if (struct.isSetO1()) {
-          struct.o1.write(oprot);
-        }
       }
 
       @Override
       public void read(org.apache.thrift.protocol.TProtocol prot, list_sentry_roles_result struct) throws org.apache.thrift.TException {
         TTupleProtocol iprot = (TTupleProtocol) prot;
-        BitSet incoming = iprot.readBitSet(2);
+        BitSet incoming = iprot.readBitSet(1);
         if (incoming.get(0)) {
           struct.success = new TListSentryRolesResponse();
           struct.success.read(iprot);
           struct.setSuccessIsSet(true);
         }
-        if (incoming.get(1)) {
-          struct.o1 = new TSentryNoSuchObjectException();
-          struct.o1.read(iprot);
-          struct.setO1IsSet(true);
-        }
       }
     }
 

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/2a1ceb57/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleAddGroupsResponse.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleAddGroupsResponse.java b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleAddGroupsResponse.java
index d52618d..0164c96 100644
--- a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleAddGroupsResponse.java
+++ b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleAddGroupsResponse.java
@@ -34,7 +34,7 @@ import org.slf4j.LoggerFactory;
 public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBase<TAlterSentryRoleAddGroupsResponse, TAlterSentryRoleAddGroupsResponse._Fields>, java.io.Serializable, Cloneable {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TAlterSentryRoleAddGroupsResponse");
 
-  private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.BOOL, (short)1);
+  private static final org.apache.thrift.protocol.TField STATUS_FIELD_DESC = new org.apache.thrift.protocol.TField("status", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
   private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
   static {
@@ -42,11 +42,11 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     schemes.put(TupleScheme.class, new TAlterSentryRoleAddGroupsResponseTupleSchemeFactory());
   }
 
-  private boolean success; // required
+  private TSentryResponseStatus status; // 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 {
-    SUCCESS((short)1, "success");
+    STATUS((short)1, "status");
 
     private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -61,8 +61,8 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
      */
     public static _Fields findByThriftId(int fieldId) {
       switch(fieldId) {
-        case 1: // SUCCESS
-          return SUCCESS;
+        case 1: // STATUS
+          return STATUS;
         default:
           return null;
       }
@@ -103,13 +103,11 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
   }
 
   // isset id assignments
-  private static final int __SUCCESS_ISSET_ID = 0;
-  private byte __isset_bitfield = 0;
   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);
-    tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.REQUIRED, 
-        new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.BOOL)));
+    tmpMap.put(_Fields.STATUS, new org.apache.thrift.meta_data.FieldMetaData("status", org.apache.thrift.TFieldRequirementType.REQUIRED, 
+        new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TSentryResponseStatus.class)));
     metaDataMap = Collections.unmodifiableMap(tmpMap);
     org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(TAlterSentryRoleAddGroupsResponse.class, metaDataMap);
   }
@@ -118,19 +116,19 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
   }
 
   public TAlterSentryRoleAddGroupsResponse(
-    boolean success)
+    TSentryResponseStatus status)
   {
     this();
-    this.success = success;
-    setSuccessIsSet(true);
+    this.status = status;
   }
 
   /**
    * Performs a deep copy on <i>other</i>.
    */
   public TAlterSentryRoleAddGroupsResponse(TAlterSentryRoleAddGroupsResponse other) {
-    __isset_bitfield = other.__isset_bitfield;
-    this.success = other.success;
+    if (other.isSetStatus()) {
+      this.status = new TSentryResponseStatus(other.status);
+    }
   }
 
   public TAlterSentryRoleAddGroupsResponse deepCopy() {
@@ -139,39 +137,39 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
 
   @Override
   public void clear() {
-    setSuccessIsSet(false);
-    this.success = false;
+    this.status = null;
   }
 
-  public boolean isSuccess() {
-    return this.success;
+  public TSentryResponseStatus getStatus() {
+    return this.status;
   }
 
-  public void setSuccess(boolean success) {
-    this.success = success;
-    setSuccessIsSet(true);
+  public void setStatus(TSentryResponseStatus status) {
+    this.status = status;
   }
 
-  public void unsetSuccess() {
-    __isset_bitfield = EncodingUtils.clearBit(__isset_bitfield, __SUCCESS_ISSET_ID);
+  public void unsetStatus() {
+    this.status = null;
   }
 
-  /** Returns true if field success is set (has been assigned a value) and false otherwise */
-  public boolean isSetSuccess() {
-    return EncodingUtils.testBit(__isset_bitfield, __SUCCESS_ISSET_ID);
+  /** Returns true if field status is set (has been assigned a value) and false otherwise */
+  public boolean isSetStatus() {
+    return this.status != null;
   }
 
-  public void setSuccessIsSet(boolean value) {
-    __isset_bitfield = EncodingUtils.setBit(__isset_bitfield, __SUCCESS_ISSET_ID, value);
+  public void setStatusIsSet(boolean value) {
+    if (!value) {
+      this.status = null;
+    }
   }
 
   public void setFieldValue(_Fields field, Object value) {
     switch (field) {
-    case SUCCESS:
+    case STATUS:
       if (value == null) {
-        unsetSuccess();
+        unsetStatus();
       } else {
-        setSuccess((Boolean)value);
+        setStatus((TSentryResponseStatus)value);
       }
       break;
 
@@ -180,8 +178,8 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
 
   public Object getFieldValue(_Fields field) {
     switch (field) {
-    case SUCCESS:
-      return Boolean.valueOf(isSuccess());
+    case STATUS:
+      return getStatus();
 
     }
     throw new IllegalStateException();
@@ -194,8 +192,8 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     }
 
     switch (field) {
-    case SUCCESS:
-      return isSetSuccess();
+    case STATUS:
+      return isSetStatus();
     }
     throw new IllegalStateException();
   }
@@ -213,12 +211,12 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     if (that == null)
       return false;
 
-    boolean this_present_success = true;
-    boolean that_present_success = true;
-    if (this_present_success || that_present_success) {
-      if (!(this_present_success && that_present_success))
+    boolean this_present_status = true && this.isSetStatus();
+    boolean that_present_status = true && that.isSetStatus();
+    if (this_present_status || that_present_status) {
+      if (!(this_present_status && that_present_status))
         return false;
-      if (this.success != that.success)
+      if (!this.status.equals(that.status))
         return false;
     }
 
@@ -229,10 +227,10 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
   public int hashCode() {
     HashCodeBuilder builder = new HashCodeBuilder();
 
-    boolean present_success = true;
-    builder.append(present_success);
-    if (present_success)
-      builder.append(success);
+    boolean present_status = true && (isSetStatus());
+    builder.append(present_status);
+    if (present_status)
+      builder.append(status);
 
     return builder.toHashCode();
   }
@@ -245,12 +243,12 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     int lastComparison = 0;
     TAlterSentryRoleAddGroupsResponse typedOther = (TAlterSentryRoleAddGroupsResponse)other;
 
-    lastComparison = Boolean.valueOf(isSetSuccess()).compareTo(typedOther.isSetSuccess());
+    lastComparison = Boolean.valueOf(isSetStatus()).compareTo(typedOther.isSetStatus());
     if (lastComparison != 0) {
       return lastComparison;
     }
-    if (isSetSuccess()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.success, typedOther.success);
+    if (isSetStatus()) {
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.status, typedOther.status);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -275,8 +273,12 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     StringBuilder sb = new StringBuilder("TAlterSentryRoleAddGroupsResponse(");
     boolean first = true;
 
-    sb.append("success:");
-    sb.append(this.success);
+    sb.append("status:");
+    if (this.status == null) {
+      sb.append("null");
+    } else {
+      sb.append(this.status);
+    }
     first = false;
     sb.append(")");
     return sb.toString();
@@ -284,11 +286,14 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
 
   public void validate() throws org.apache.thrift.TException {
     // check for required fields
-    if (!isSetSuccess()) {
-      throw new org.apache.thrift.protocol.TProtocolException("Required field 'success' is unset! Struct:" + toString());
+    if (!isSetStatus()) {
+      throw new org.apache.thrift.protocol.TProtocolException("Required field 'status' is unset! Struct:" + toString());
     }
 
     // check for sub-struct validity
+    if (status != null) {
+      status.validate();
+    }
   }
 
   private void writeObject(java.io.ObjectOutputStream out) throws java.io.IOException {
@@ -301,8 +306,6 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
 
   private void readObject(java.io.ObjectInputStream in) throws java.io.IOException, ClassNotFoundException {
     try {
-      // it doesn't seem like you should have to do this, but java serialization is wacky, and doesn't call the default constructor.
-      __isset_bitfield = 0;
       read(new org.apache.thrift.protocol.TCompactProtocol(new org.apache.thrift.transport.TIOStreamTransport(in)));
     } catch (org.apache.thrift.TException te) {
       throw new java.io.IOException(te);
@@ -327,10 +330,11 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
           break;
         }
         switch (schemeField.id) {
-          case 1: // SUCCESS
-            if (schemeField.type == org.apache.thrift.protocol.TType.BOOL) {
-              struct.success = iprot.readBool();
-              struct.setSuccessIsSet(true);
+          case 1: // STATUS
+            if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
+              struct.status = new TSentryResponseStatus();
+              struct.status.read(iprot);
+              struct.setStatusIsSet(true);
             } else { 
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
             }
@@ -348,9 +352,11 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
       struct.validate();
 
       oprot.writeStructBegin(STRUCT_DESC);
-      oprot.writeFieldBegin(SUCCESS_FIELD_DESC);
-      oprot.writeBool(struct.success);
-      oprot.writeFieldEnd();
+      if (struct.status != null) {
+        oprot.writeFieldBegin(STATUS_FIELD_DESC);
+        struct.status.write(oprot);
+        oprot.writeFieldEnd();
+      }
       oprot.writeFieldStop();
       oprot.writeStructEnd();
     }
@@ -368,14 +374,15 @@ public class TAlterSentryRoleAddGroupsResponse implements org.apache.thrift.TBas
     @Override
     public void write(org.apache.thrift.protocol.TProtocol prot, TAlterSentryRoleAddGroupsResponse struct) throws org.apache.thrift.TException {
       TTupleProtocol oprot = (TTupleProtocol) prot;
-      oprot.writeBool(struct.success);
+      struct.status.write(oprot);
     }
 
     @Override
     public void read(org.apache.thrift.protocol.TProtocol prot, TAlterSentryRoleAddGroupsResponse struct) throws org.apache.thrift.TException {
       TTupleProtocol iprot = (TTupleProtocol) prot;
-      struct.success = iprot.readBool();
-      struct.setSuccessIsSet(true);
+      struct.status = new TSentryResponseStatus();
+      struct.status.read(iprot);
+      struct.setStatusIsSet(true);
     }
   }
 

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/2a1ceb57/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleDeleteGroupsResponse.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleDeleteGroupsResponse.java b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleDeleteGroupsResponse.java
index f11808f..5720773 100644
--- a/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleDeleteGroupsResponse.java
+++ b/sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/policystore/api/TAlterSentryRoleDeleteGroupsResponse.java
@@ -34,7 +34,7 @@ import org.slf4j.LoggerFactory;
 public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.TBase<TAlterSentryRoleDeleteGroupsResponse, TAlterSentryRoleDeleteGroupsResponse._Fields>, java.io.Serializable, Cloneable {
   private static final org.apache.thrift.protocol.TStruct STRUCT_DESC = new org.apache.thrift.protocol.TStruct("TAlterSentryRoleDeleteGroupsResponse");
 
-  private static final org.apache.thrift.protocol.TField SUCCESS_FIELD_DESC = new org.apache.thrift.protocol.TField("success", org.apache.thrift.protocol.TType.BOOL, (short)1);
+  private static final org.apache.thrift.protocol.TField STATUS_FIELD_DESC = new org.apache.thrift.protocol.TField("status", org.apache.thrift.protocol.TType.STRUCT, (short)1);
 
   private static final Map<Class<? extends IScheme>, SchemeFactory> schemes = new HashMap<Class<? extends IScheme>, SchemeFactory>();
   static {
@@ -42,11 +42,11 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     schemes.put(TupleScheme.class, new TAlterSentryRoleDeleteGroupsResponseTupleSchemeFactory());
   }
 
-  private boolean success; // required
+  private TSentryResponseStatus status; // 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 {
-    SUCCESS((short)1, "success");
+    STATUS((short)1, "status");
 
     private static final Map<String, _Fields> byName = new HashMap<String, _Fields>();
 
@@ -61,8 +61,8 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
      */
     public static _Fields findByThriftId(int fieldId) {
       switch(fieldId) {
-        case 1: // SUCCESS
-          return SUCCESS;
+        case 1: // STATUS
+          return STATUS;
         default:
           return null;
       }
@@ -103,13 +103,11 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
   }
 
   // isset id assignments
-  private static final int __SUCCESS_ISSET_ID = 0;
-  private byte __isset_bitfield = 0;
   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);
-    tmpMap.put(_Fields.SUCCESS, new org.apache.thrift.meta_data.FieldMetaData("success", org.apache.thrift.TFieldRequirementType.REQUIRED, 
-        new org.apache.thrift.meta_data.FieldValueMetaData(org.apache.thrift.protocol.TType.BOOL)));
+    tmpMap.put(_Fields.STATUS, new org.apache.thrift.meta_data.FieldMetaData("status", org.apache.thrift.TFieldRequirementType.REQUIRED, 
+        new org.apache.thrift.meta_data.StructMetaData(org.apache.thrift.protocol.TType.STRUCT, TSentryResponseStatus.class)));
     metaDataMap = Collections.unmodifiableMap(tmpMap);
     org.apache.thrift.meta_data.FieldMetaData.addStructMetaDataMap(TAlterSentryRoleDeleteGroupsResponse.class, metaDataMap);
   }
@@ -118,19 +116,19 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
   }
 
   public TAlterSentryRoleDeleteGroupsResponse(
-    boolean success)
+    TSentryResponseStatus status)
   {
     this();
-    this.success = success;
-    setSuccessIsSet(true);
+    this.status = status;
   }
 
   /**
    * Performs a deep copy on <i>other</i>.
    */
   public TAlterSentryRoleDeleteGroupsResponse(TAlterSentryRoleDeleteGroupsResponse other) {
-    __isset_bitfield = other.__isset_bitfield;
-    this.success = other.success;
+    if (other.isSetStatus()) {
+      this.status = new TSentryResponseStatus(other.status);
+    }
   }
 
   public TAlterSentryRoleDeleteGroupsResponse deepCopy() {
@@ -139,39 +137,39 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
 
   @Override
   public void clear() {
-    setSuccessIsSet(false);
-    this.success = false;
+    this.status = null;
   }
 
-  public boolean isSuccess() {
-    return this.success;
+  public TSentryResponseStatus getStatus() {
+    return this.status;
   }
 
-  public void setSuccess(boolean success) {
-    this.success = success;
-    setSuccessIsSet(true);
+  public void setStatus(TSentryResponseStatus status) {
+    this.status = status;
   }
 
-  public void unsetSuccess() {
-    __isset_bitfield = EncodingUtils.clearBit(__isset_bitfield, __SUCCESS_ISSET_ID);
+  public void unsetStatus() {
+    this.status = null;
   }
 
-  /** Returns true if field success is set (has been assigned a value) and false otherwise */
-  public boolean isSetSuccess() {
-    return EncodingUtils.testBit(__isset_bitfield, __SUCCESS_ISSET_ID);
+  /** Returns true if field status is set (has been assigned a value) and false otherwise */
+  public boolean isSetStatus() {
+    return this.status != null;
   }
 
-  public void setSuccessIsSet(boolean value) {
-    __isset_bitfield = EncodingUtils.setBit(__isset_bitfield, __SUCCESS_ISSET_ID, value);
+  public void setStatusIsSet(boolean value) {
+    if (!value) {
+      this.status = null;
+    }
   }
 
   public void setFieldValue(_Fields field, Object value) {
     switch (field) {
-    case SUCCESS:
+    case STATUS:
       if (value == null) {
-        unsetSuccess();
+        unsetStatus();
       } else {
-        setSuccess((Boolean)value);
+        setStatus((TSentryResponseStatus)value);
       }
       break;
 
@@ -180,8 +178,8 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
 
   public Object getFieldValue(_Fields field) {
     switch (field) {
-    case SUCCESS:
-      return Boolean.valueOf(isSuccess());
+    case STATUS:
+      return getStatus();
 
     }
     throw new IllegalStateException();
@@ -194,8 +192,8 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     }
 
     switch (field) {
-    case SUCCESS:
-      return isSetSuccess();
+    case STATUS:
+      return isSetStatus();
     }
     throw new IllegalStateException();
   }
@@ -213,12 +211,12 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     if (that == null)
       return false;
 
-    boolean this_present_success = true;
-    boolean that_present_success = true;
-    if (this_present_success || that_present_success) {
-      if (!(this_present_success && that_present_success))
+    boolean this_present_status = true && this.isSetStatus();
+    boolean that_present_status = true && that.isSetStatus();
+    if (this_present_status || that_present_status) {
+      if (!(this_present_status && that_present_status))
         return false;
-      if (this.success != that.success)
+      if (!this.status.equals(that.status))
         return false;
     }
 
@@ -229,10 +227,10 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
   public int hashCode() {
     HashCodeBuilder builder = new HashCodeBuilder();
 
-    boolean present_success = true;
-    builder.append(present_success);
-    if (present_success)
-      builder.append(success);
+    boolean present_status = true && (isSetStatus());
+    builder.append(present_status);
+    if (present_status)
+      builder.append(status);
 
     return builder.toHashCode();
   }
@@ -245,12 +243,12 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     int lastComparison = 0;
     TAlterSentryRoleDeleteGroupsResponse typedOther = (TAlterSentryRoleDeleteGroupsResponse)other;
 
-    lastComparison = Boolean.valueOf(isSetSuccess()).compareTo(typedOther.isSetSuccess());
+    lastComparison = Boolean.valueOf(isSetStatus()).compareTo(typedOther.isSetStatus());
     if (lastComparison != 0) {
       return lastComparison;
     }
-    if (isSetSuccess()) {
-      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.success, typedOther.success);
+    if (isSetStatus()) {
+      lastComparison = org.apache.thrift.TBaseHelper.compareTo(this.status, typedOther.status);
       if (lastComparison != 0) {
         return lastComparison;
       }
@@ -275,8 +273,12 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     StringBuilder sb = new StringBuilder("TAlterSentryRoleDeleteGroupsResponse(");
     boolean first = true;
 
-    sb.append("success:");
-    sb.append(this.success);
+    sb.append("status:");
+    if (this.status == null) {
+      sb.append("null");
+    } else {
+      sb.append(this.status);
+    }
     first = false;
     sb.append(")");
     return sb.toString();
@@ -284,11 +286,14 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
 
   public void validate() throws org.apache.thrift.TException {
     // check for required fields
-    if (!isSetSuccess()) {
-      throw new org.apache.thrift.protocol.TProtocolException("Required field 'success' is unset! Struct:" + toString());
+    if (!isSetStatus()) {
+      throw new org.apache.thrift.protocol.TProtocolException("Required field 'status' is unset! Struct:" + toString());
     }
 
     // check for sub-struct validity
+    if (status != null) {
+      status.validate();
+    }
   }
 
   private void writeObject(java.io.ObjectOutputStream out) throws java.io.IOException {
@@ -301,8 +306,6 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
 
   private void readObject(java.io.ObjectInputStream in) throws java.io.IOException, ClassNotFoundException {
     try {
-      // it doesn't seem like you should have to do this, but java serialization is wacky, and doesn't call the default constructor.
-      __isset_bitfield = 0;
       read(new org.apache.thrift.protocol.TCompactProtocol(new org.apache.thrift.transport.TIOStreamTransport(in)));
     } catch (org.apache.thrift.TException te) {
       throw new java.io.IOException(te);
@@ -327,10 +330,11 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
           break;
         }
         switch (schemeField.id) {
-          case 1: // SUCCESS
-            if (schemeField.type == org.apache.thrift.protocol.TType.BOOL) {
-              struct.success = iprot.readBool();
-              struct.setSuccessIsSet(true);
+          case 1: // STATUS
+            if (schemeField.type == org.apache.thrift.protocol.TType.STRUCT) {
+              struct.status = new TSentryResponseStatus();
+              struct.status.read(iprot);
+              struct.setStatusIsSet(true);
             } else { 
               org.apache.thrift.protocol.TProtocolUtil.skip(iprot, schemeField.type);
             }
@@ -348,9 +352,11 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
       struct.validate();
 
       oprot.writeStructBegin(STRUCT_DESC);
-      oprot.writeFieldBegin(SUCCESS_FIELD_DESC);
-      oprot.writeBool(struct.success);
-      oprot.writeFieldEnd();
+      if (struct.status != null) {
+        oprot.writeFieldBegin(STATUS_FIELD_DESC);
+        struct.status.write(oprot);
+        oprot.writeFieldEnd();
+      }
       oprot.writeFieldStop();
       oprot.writeStructEnd();
     }
@@ -368,14 +374,15 @@ public class TAlterSentryRoleDeleteGroupsResponse implements org.apache.thrift.T
     @Override
     public void write(org.apache.thrift.protocol.TProtocol prot, TAlterSentryRoleDeleteGroupsResponse struct) throws org.apache.thrift.TException {
       TTupleProtocol oprot = (TTupleProtocol) prot;
-      oprot.writeBool(struct.success);
+      struct.status.write(oprot);
     }
 
     @Override
     public void read(org.apache.thrift.protocol.TProtocol prot, TAlterSentryRoleDeleteGroupsResponse struct) throws org.apache.thrift.TException {
       TTupleProtocol iprot = (TTupleProtocol) prot;
-      struct.success = iprot.readBool();
-      struct.setSuccessIsSet(true);
+      struct.status = new TSentryResponseStatus();
+      struct.status.read(iprot);
+      struct.setStatusIsSet(true);
     }
   }