优化SQLBinaryOpExpr的括号解析逻辑 #5847 #5855
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
优化SQLBinaryOpExpr的括号解析逻辑 alibaba#5847
根据单测优化解析逻辑
优化逻辑，调整单测断言
继续优化sql生成逻辑和单测断言
还有70多个单测没搞定
持续优化括号相关逻辑,还剩58个失败用例
继续优化解析括号的逻辑，单测还剩58个不成功
持续优化括号解析和输出逻辑，还剩29个单测未通过
优化发现是注释生成问题，单测还剩21个
终于优化完括号解析和输出逻辑
lizongbo
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
优化SQLBinaryOpExpr的括号解析逻辑 #5847 调整了解析逻辑，顺便修复以前存在问题的单测