[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388579721 ## File path: core/src/test/resources/sql/sub-query.iq ## @@ -831,7 +831,7 @@

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388573407 ## File path: core/src/test/java/org/apache/calcite/test/LatticeTest.java ## @@ -677,10 +677,10 @@ private

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388573407 ## File path: core/src/test/java/org/apache/calcite/test/LatticeTest.java ## @@ -677,10 +677,10 @@ private

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388570453 ## File path: core/src/test/resources/sql/sub-query.iq ## @@ -831,7 +831,7 @@

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388570453 ## File path: core/src/test/resources/sql/sub-query.iq ## @@ -831,7 +831,7 @@

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388218456 ## File path: core/src/test/resources/sql/sub-query.iq ## @@ -831,7 +831,7 @@

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-05 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r388213305 ## File path: core/src/test/java/org/apache/calcite/test/LatticeTest.java ## @@ -677,10 +677,10 @@ private

[GitHub] [calcite] vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-04 Thread GitBox
vlsi commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r387935235 ## File path: core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java ## @@ -87,36 +85,11