fix(authorization-ranger):the descr field of ranger table x_group is …

…not null (#4975) ### What changes were proposed in this pull request? When calling the ranger CREATE_GROUP api, the desc field cannot be null ### Why are the changes needed? When calling the ranger CREATE_GROUP api, ranger response is error. ### Does this PR introduce _any_ user-facing change? ### How was this patch tested? yes, by hand. --------- Co-authored-by: yangyx <[email protected]>
apache · Sep 27, 2024 · f38cba7 · f38cba7
1 parent 174f74d
commit f38cba7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/...er/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java b/...er/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java
@@ -383,7 +383,8 @@ public Boolean onUserAcquired(User user) throws RuntimeException {
 
   @Override
   public Boolean onGroupAdded(Group group) throws RuntimeException {
-    return rangerClient.createGroup(VXGroup.builder().withName(group.name()).build());
+    return rangerClient.createGroup(
+        VXGroup.builder().withName(group.name()).withDescription(group.name()).build());
   }
 
   @Override

diff --git a/...ion-ranger/src/main/java/org/apache/gravitino/authorization/ranger/reference/VXGroup.java b/...ion-ranger/src/main/java/org/apache/gravitino/authorization/ranger/reference/VXGroup.java
@@ -91,6 +91,11 @@ public Builder withName(String name) {
       return this;
     }
 
+    public Builder withDescription(String description) {
+      vxGroup.description = description;
+      return this;
+    }
+
     public VXGroup build() {
       return vxGroup;
     }