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

优化类AbstractMapper，将for循环拼接的SQL语句优化为Lambda表达式拼接，使得代码逻辑更加清晰简洁 #12911

Closed
20144101037 opened this issue Dec 3, 2024 · 2 comments · Fixed by #12914
Closed

优化类AbstractMapper，将for循环拼接的SQL语句优化为Lambda表达式拼接，使得代码逻辑更加清晰简洁 #12911

20144101037 opened this issue Dec 3, 2024 · 2 comments · Fixed by #12914
Labels
area/Config plugin

Comments

@20144101037
Copy link
Contributor

Issue Description

Type: AbstractMapper类代码优化

Describe what happened (or what feature you want)

优化类AbstractMapper，将多条for循环拼接的SQL语句优化为Lambda表达式拼接。

Describe what you expected to happen

使得代码逻辑更加清晰简洁

How to reproduce it (as minimally and precisely as possible)

Tell us your environment

nacos最新的develop分支

Anything else we need to know?
@KomachiSion
Copy link
Collaborator

Welcome PR

@20144101037
Copy link
Contributor Author

OK，Thank you

20144101037 added a commit to 20144101037/nacos that referenced this issue Dec 6, 2024
@20144101037
[ISSUE alibaba#12911] resolve code conflict
a8fb72f
KomachiSion pushed a commit that referenced this issue Dec 6, 2024
@20144101037
【ISSUE #12911】将for循环拼接的SQL语句改为Lambda表达式拼接，使得代码简洁清晰，并添加相关测试用例 (#12914)
cb3685d 
* feat(优化抽象类AbstractMapper) 将for循环拼接的SQL语句改为Lambda表达式拼接，使得代码简洁清晰，并添加相关测试用例

* [ISSUE #12914] 优化抽象类AbstractMapper，将for循环拼接的SQL语句改为Lambda表达式拼接，使得代码简洁清晰，并添加相关测试用例

* [ISSUE #12911] resolve code conflict
@KomachiSion KomachiSion linked a pull request Dec 6, 2024 that will close this issue
【ISSUE #12911】将for循环拼接的SQL语句改为Lambda表达式拼接，使得代码简洁清晰，并添加相关测试用例 #12914
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/Config plugin
Projects
None yet
Milestone
No milestone
2 participants
@20144101037 @KomachiSion