lasdf1234 commented on PR #12288:
URL: https://github.com/apache/gravitino/pull/12288#issuecomment-5140744535
> > Why is the input parameter of addUsers a List users? Your scenario
should not include externalId. Let's first create a simpler version without
externalId.
> > You can modify the code according to this logic, and then I will
continue to review it. Thank you. (The first PR has many details to be
discussed and may need to be revised many times. The subsequent PRs will be
merged quickly.)
>
> Thanks for the detailed suggestion.
>
> Regarding the `addUsers` input, the current request body follows the
approved design doc. In section 3.1, the bulk add users API is defined as `POST
/api/bulk/metalakes/{metalake}/users/add`, and the request body is specified as
a `users` array whose items contain `name`, `externalId`, and `enabled`:
>
> ```json
> {
> "users": [
> { "name": "alice", "externalId": "ext-alice", "enabled": true },
> { "name": "bob", "enabled": true },
> { "name": "alice" }
> ]
> }
> ```
I have already corrected the comment.
--
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]
