[GitHub] drill pull request #1204: DRILL-6318

2018-04-10 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1204#discussion_r180538860 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -74,9 +74,12 @@ public boolean matches(RelO

[GitHub] drill pull request #1204: DRILL-6318

2018-04-10 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1204#discussion_r180539160 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -89,14 +92,17 @@ public void onMatch(RelOpt

[GitHub] drill pull request #1204: DRILL-6318

2018-04-10 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/1204#discussion_r180539313 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/flatten/TestFlattenPlanning.java --- @@ -66,8 +66,11 @@ public void testPushFilter

[GitHub] drill pull request #1204: DRILL-6318

2018-04-10 Thread oleg-zinovev
GitHub user oleg-zinovev opened a pull request: https://github.com/apache/drill/pull/1204 DRILL-6318 Test for DRILL-6099 was changed due it's incorrect behavior You can merge this pull request into a Git repository by running: $ git pull https://github.com/oleg-zinovev/drill d