[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-25 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r205250070 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRel

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-25 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r205250094 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/store/parquet/TestParque

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-25 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r205249991 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillFil

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-25 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r205250018 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPus

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204952967 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/RuleInstance.jav

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204952905 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPus

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204952377 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/RuleInstance.jav

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204951832 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRel

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204951732 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRel

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204951637 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRel

[GitHub] gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects

2018-07-24 Thread GitBox
gparai commented on a change in pull request #1372: DRILL-6589: Push transitive closure predicates past aggregates/projects URL: https://github.com/apache/drill/pull/1372#discussion_r204951494 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestAggregateFun