soar icon indicating copy to clipboard operation
soar copied to clipboard

rewrite having skip calc expr

Open kosmgco opened this issue 4 years ago • 0 comments

What problem does this PR solve?

SOAR对于having语句的rewrite处理聚合函数存在问题 #63

What is changed and how it works?

RewriteHaving 时判断 having 的各个条件是否是计算列,如果为计算列则不替换到 where 子句

Check List

Tests

  • Unit test go test -run ^TestRewriteHaving$ github.com/XiaoMi/soar/ast -test.v -count=1
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

kosmgco avatar Aug 03 '21 06:08 kosmgco