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.


---

Reply via email to