[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1096 ---

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171711326 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171708636 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171708439 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171708410 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171708384 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-28 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171479641 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-28 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171480227 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-28 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171478117 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-28 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171478085 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -224,4 +226,64 @@ public Void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171086924 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r171085780 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-02 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r165788152 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-02-02 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r165791415 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -55,18 +62,21 @@ public void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-01-29 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r164593749 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -121,4 +132,50 @@ protected void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-01-29 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r164585670 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -121,4 +132,50 @@ protected void

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-01-22 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r163048747 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/PlannerPhase.java --- @@ -341,7 +346,7 @@ static RuleSet

[GitHub] drill pull request #1096: DRILL - 6099 : Push limit past flatten(project) wi...

2018-01-19 Thread gparai
GitHub user gparai opened a pull request: https://github.com/apache/drill/pull/1096 DRILL - 6099 : Push limit past flatten(project) without pushdown into scan @amansinha100 @chunhui-shi can you please review this PR? Thanks! You can merge this pull request into a Git repository by