Return an empty Privilege Set instead of an error.

danielbwatson · danielbwatson · commit 2aab99f45121 · 2016-05-12T15:23:29.000-07:00
diff --git a/metacat-thrift/src/main/java/com/netflix/metacat/thrift/CatalogThriftHiveMetastore.java b/metacat-thrift/src/main/java/com/netflix/metacat/thrift/CatalogThriftHiveMetastore.java
@@ -874,9 +874,10 @@ public GetPrincipalsInRoleResponse get_principals_in_role(GetPrincipalsInRoleReq
     }
 
     @Override
-    public PrincipalPrivilegeSet get_privilege_set(HiveObjectRef hiveObject, String user_name, List<String> group_names)
+    public PrincipalPrivilegeSet get_privilege_set(HiveObjectRef hiveObject, String userName, List<String> groupNames)
             throws TException {
-        throw unimplemented("get_privilege_set", new Object[] { hiveObject, user_name, group_names });
+        return requestWrapper("get_privilege_set", new Object[] { hiveObject, userName, groupNames },
+                PrincipalPrivilegeSet::new);
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -874,9 +874,10 @@ public GetPrincipalsInRoleResponse get_principals_in_role(GetPrincipalsInRoleReq`
`874`	`874`	`}`
`875`	`875`
`876`	`876`	`@Override`
`877`		`- public PrincipalPrivilegeSet get_privilege_set(HiveObjectRef hiveObject, String user_name, List<String> group_names)`
	`877`	`+ public PrincipalPrivilegeSet get_privilege_set(HiveObjectRef hiveObject, String userName, List<String> groupNames)`
`878`	`878`	`throws TException {`
`879`		`- throw unimplemented("get_privilege_set", new Object[] { hiveObject, user_name, group_names });`
	`879`	`+ return requestWrapper("get_privilege_set", new Object[] { hiveObject, userName, groupNames },`
	`880`	`+ PrincipalPrivilegeSet::new);`
`880`	`881`	`}`
`881`	`882`
`882`	`883`	`@Override`