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.

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

优化SQLBinaryOpExpr的括号解析逻辑 #5847 #5855

Merged
merged 20 commits into from
May 3, 2024

Conversation

lizongbo
Copy link
Collaborator

@lizongbo lizongbo commented Apr 20, 2024

优化SQLBinaryOpExpr的括号解析逻辑 #5847 调整了解析逻辑,顺便修复以前存在问题的单测

优化SQLBinaryOpExpr的括号解析逻辑 alibaba#5847
根据单测优化解析逻辑
优化逻辑,调整单测断言
持续调整单测断言
持续调整单测
继续优化sql生成逻辑和单测断言
继续优化逻辑和断言
继续调整单测
持续调整单测
还有70多个单测没搞定
持续优化括号相关逻辑,还剩58个失败用例
继续优化解析括号的逻辑,单测还剩58个不成功
继续优化括号逻辑,还剩44个单测未验证和调整
持续优化括号解析和输出逻辑,还剩29个单测未通过
优化发现是注释生成问题,单测还剩21个
持续优化代码,还剩8个单测未通过
终于优化完括号解析和输出逻辑
@lizongbo lizongbo changed the title 优化SQLBinaryOpExpr的括号解析逻辑 #5847 还没改完测试用例 优化SQLBinaryOpExpr的括号解析逻辑 #5847 May 3, 2024
@lizongbo lizongbo merged commit 0c1332a into alibaba:master May 3, 2024
5 checks passed
handsomestWei added a commit to handsomestWei/mybatis-mapper2sql that referenced this pull request Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant