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 testPushFilterPastProjectWithFlattenNeg()
throws Exception {
@Test // DRILL-6099 : push limit past flatten(project)
public void testLimitPushdownPastFlatten() throws Exception {
final String query = "select rownum, flatten(complex) comp from
cp.`store/json/test_flatten_mappify2.json` limit 1";
- final String[] expectedPatterns =
{".*Limit\\(fetch=\\[1\\]\\).*",".*Flatten.*",".*Limit\\(fetch=\\[1\\]\\).*"};
- final String[] excludedPatterns = null;
+ //DRILL-6318 : limit should not push past flatten(project)
+ //P.S. Where was an error in this pattern. Even then Limit missing
after Flatten it matches to plan
--- End diff --
Remove the P.S. and commented out pattern.
---
