Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Server Config Operation Interface Refactor(Optimize) #12994

Open
shiyiyue1102 opened this issue Dec 26, 2024 · 1 comment · May be fixed by #13002
Open

Server Config Operation Interface Refactor(Optimize) #12994

shiyiyue1102 opened this issue Dec 26, 2024 · 1 comment · May be fixed by #13002
Labels
kind/enhancement Category issues or prs related to enhancement. kind/refactor

Comments

@shiyiyue1102
Copy link
Collaborator

1.Uniform config update operations(such as config update,delete, insert and related batch operations etc... ) to ConfigOperationService
2. change aspects, such as CapacityManagementAspect,RequestLogAspect ,ConfigChangeAspect ,from controller（http） and handler(grpc remote) interface to protocol-less ConfigOperationService interface
3.
@shiyiyue1102 shiyiyue1102 changed the title Config Operation Interface Refactor(Optimize) Server Config Operation Interface Refactor(Optimize) Dec 26, 2024
@KomachiSion KomachiSion added kind/enhancement Category issues or prs related to enhancement. kind/refactor labels Dec 30, 2024
@MatthewAden MatthewAden linked a pull request Dec 30, 2024 that will close this issue
Open
5 tasks
@MatthewAden
Copy link
Contributor

ConfigChangeAspect这个切面类是不是维持现状比较好，ConfigChangeAspect目前支持配置变更插件针对不同协议的配置变更进行差异化处理，如果统一成无协议的话，这种差异化处理的能力就会丧失

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/enhancement Category issues or prs related to enhancement. kind/refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE#12994]Optimize config operation. MatthewAden/nacos
3 participants
@shiyiyue1102 @KomachiSion @MatthewAden