lasdf1234 commented on code in PR #11848:
URL: https://github.com/apache/gravitino/pull/11848#discussion_r3511265732


##########
core/src/main/java/org/apache/gravitino/authorization/AccessControlDispatcher.java:
##########
@@ -51,6 +51,21 @@ public interface AccessControlDispatcher {
   User addUser(String metalake, String user)
       throws UserAlreadyExistsException, NoSuchMetalakeException;
 
+  /**
+   * Adds a new User with an external identifier and enabled state.
+   *
+   * @param metalake The Metalake of the User.
+   * @param user The name of the User.
+   * @param externalId The external identifier from an upstream identity 
system.
+   * @param enabled Whether the User is enabled.
+   * @return The added User instance.
+   * @throws UserAlreadyExistsException If a User with the same name already 
exists.
+   * @throws NoSuchMetalakeException If the Metalake with the given name does 
not exist.
+   * @throws RuntimeException If adding the User encounters storage issues.
+   */
+  User addUser(String metalake, String user, String externalId, boolean 
enabled)

Review Comment:
   Got resolved.
   Old method add parameters.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to