Skip to content

feat(mac): sod mandatory and mac[manual-only] #35121

Merged
guanshengliang merged 93 commits intomainfrom
feat/TD-6670071929-main
Apr 22, 2026
Merged

feat(mac): sod mandatory and mac[manual-only] #35121
guanshengliang merged 93 commits intomainfrom
feat/TD-6670071929-main

Conversation

@kailixu
Copy link
Copy Markdown
Contributor

@kailixu kailixu commented Apr 13, 2026

Description

Issue(s)

  • Close/close/Fix/fix/Resolve/resolve: Issue Link

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

kailixu added 30 commits February 18, 2026 13:40
@kailixu
feat(mac): add security_level for db/table
2a1fe31
@kailixu
feat(mac): security_level for user
ab72a77
@kailixu
feat(mac): security_level for db/vg
21e6c4e
@kailixu
feat(mac): security_level for user
7e90147
@kailixu
feat(mac): security_level for user
f04ae35
@kailixu
feat(mac): add test case
5ea85f0
@kailixu
feat(mac): show create db/stable
4713666
@kailixu
feat(mac): support allow_drop when create db
bb87805
@kailixu
feat(mac): support allow_drop when create db
09c7d31
@kailixu
feat(mac): support show security_polices
efb4f4a
@kailixu
feat(mac): support show security_policies
d696771
@kailixu
feat(mac): check show security_policies
e600fa6
@kailixu
feat(mac): alter cluster sod mandatory
fabd40a
@kailixu
feat(mac): check prerequisites for sod mandatory
72d1dd4
@kailixu
feat(mac): alter cluster sod mandatory
25d3d47
@kailixu
feat(mac): alter cluster sod mandatory
53c2543
@kailixu
feat(mac): process user disable
b88b6fc
@kailixu
feat(mac): process user disable
01124c1
@kailixu
feat(mac): enforce SoD by command line
0ea7806
@kailixu
feat(mac): enforce SoD by command line
abeddc3
@kailixu
feat(mac): enforce SoD by command line and SQL command
9900d79
@kailixu
feat(mac): dispatch sodInitial to client
9e0831b
@kailixu
feat(mac): enforce SoD mandatory
95e4c7d
@kailixu
feat(sod): sod mandatory
8007960
@kailixu
feat(sod): sod mandatory
2c0d6f3
@kailixu
feat(sod): sod mandatory
1f8b417
@kailixu
feat(sod): sod mandatory optimization
4981d40
@kailixu
feat(sod): realtime check in sodInitial phase when grant system manag…
87cdeff 
…ement roles
@kailixu
feat(sod): ensure 3 management roles still satisfied after disable/dr…
3b3eca4 
…op/revoke
@kailixu
feat(sod): ensure 3 management roles still satisfied after disable/dr…
1e454a7 
…op/revoke
Copilot AI reviewed Apr 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/libs/executor/src/sysscanoperator.c
Copilot AI review requested due to automatic review settings April 21, 2026 11:05
Copilot AI reviewed Apr 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/libs/executor/src/sysscanoperator.c Outdated
Comment thread source/libs/parser/src/parTranslater.c
Copilot AI review requested due to automatic review settings April 21, 2026 11:56
Copilot AI reviewed Apr 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/libs/parser/src/parAstCreater.c Outdated
xiao-77
xiao-77 reviewed Apr 22, 2026
View reviewed changes
Comment thread source/dnode/mnode/impl/src/mndStb.c
#define STB_VER_SUPPORT_OWNER 4
#define STB_VER_NUMBER STB_VER_SUPPORT_OWNER
#define STB_RESERVE_SIZE 55
#define STB_RESERVE_SIZE 51
Copy link
Copy Markdown
Contributor

@xiao-77 xiao-77 Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这些 sdb 的修改要不要在3.0上占位？

Copy link
Copy Markdown
Contributor Author

@kailixu kailixu Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

需要将 main merge 到 3.0.

Copilot AI review requested due to automatic review settings April 22, 2026 03:15
Copilot AI reviewed Apr 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/libs/parser/src/parAstCreater.c
Copilot AI review requested due to automatic review settings April 22, 2026 04:06
Copilot AI reviewed Apr 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread include/util/tdef.h
Copilot AI review requested due to automatic review settings April 22, 2026 12:20
Copilot AI reviewed Apr 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 93 out of 93 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/dnode/mnode/impl/src/mndUser.c
@guanshengliang guanshengliang merged commit fde63d1 into main Apr 22, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

Copilot code review Copilot Copilot left review comments

@guanshengliang guanshengliang guanshengliang approved these changes

@xiao-77 xiao-77 xiao-77 approved these changes

@zitsen zitsen Awaiting requested review from zitsen zitsen is a code owner

@dapan1121 dapan1121 Awaiting requested review from dapan1121 dapan1121 is a code owner

@feici02 feici02 Awaiting requested review from feici02 feici02 is a code owner

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

team engine

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kailixu @guanshengliang @xiao-77