[GitHub] [calcite] vlsi commented on a change in pull request #2144: [CALCITE-4237] Sarg with single IS NULL operand causes AssertionError

2020-09-09 Thread GitBox
vlsi commented on a change in pull request #2144: URL: https://github.com/apache/calcite/pull/2144#discussion_r485468654 ## File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java ## @@ -2690,7 +2691,14 @@ int complexity() { } @Override public

[GitHub] [calcite] vlsi commented on a change in pull request #2144: [CALCITE-4237] Sarg with single IS NULL operand causes AssertionError

2020-09-09 Thread GitBox
vlsi commented on a change in pull request #2144: URL: https://github.com/apache/calcite/pull/2144#discussion_r485449641 ## File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java ## @@ -2690,7 +2691,14 @@ int complexity() { } @Override public

[GitHub] [calcite] vlsi commented on a change in pull request #2144: [CALCITE-4237] Sarg with single IS NULL operand causes AssertionError

2020-09-09 Thread GitBox
vlsi commented on a change in pull request #2144: URL: https://github.com/apache/calcite/pull/2144#discussion_r485439222 ## File path: plus/src/test/java/org/apache/calcite/adapter/tpcds/TpcdsTest.java ## @@ -367,15 +366,13 @@ protected void foo(CalciteAssert.AssertThat with,