lasdf1234 opened a new issue, #12493:
URL: https://github.com/apache/gravitino/issues/12493
### Describe the proposal
Built-in IdP groups currently store only `group_name`. Operators have no
place to record a short description (for example what the group is for, or who
owns it) when creating a group via `POST /api/idp/groups`.
Add an optional **comment** on local IdP groups, aligned with existing
comment columns such as `tag_comment` / `column_comment`.
### Proposed approach
- Persist `group_comment VARCHAR(256) DEFAULT ''` on `idp_group_meta`.
- Expose optional REST JSON field `comment` on group create and get.
- Omit on create → store empty string.
- Existing rows get `''` via column default / 1.3.0 → 2.0.0 upgrade.
- Schema: add the column on **2.0.0** CREATE scripts and **1.3.0 → 2.0.0**
ALTER (MySQL / PostgreSQL / H2). Do not change released 1.3.0 schema.
- Out of scope for this issue: `PUT /api/idp/groups/{group}` (update
comment). Create writes it; GET returns it.
### Task list
- [ ] Relational schema + upgrade scripts
- [ ] Storage / domain / REST (`AddGroupRequest`, `IdpGroupDTO`)
- [ ] OpenAPI + local users/groups docs
- [ ] Unit tests for create/get round-trip
--
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]
