You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2020/05/27 14:22:09 UTC

[GitHub] [drill] bozzzzo opened a new pull request #2083: DRILL-7722: Add reproducer unit test

bozzzzo opened a new pull request #2083:
URL: https://github.com/apache/drill/pull/2083


   # [DRILL-7722](https://issues.apache.org/jira/browse/DRILL-7722): Add reproducer unit test
   
   ## Description
   
   Add a unit test that demonstrates that a view with UNNEST cannot be sucessfully queried.
   
   ## Documentation
   Fixing the bug will make documentation as-is correct
   
   ## Testing
   The added test fails. 
   
   A slightly modified test that does not use the view, but directly executes the view query, passes:
   ```
   $ git diff
   diff --git a/exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestViewSupport.java b/exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestViewSupport.java
   index 1a92e192f..7c317f427 100644
   --- a/exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestViewSupport.java
   +++ b/exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestViewSupport.java
   @@ -857,7 +857,8 @@ public class TestViewSupport extends TestBaseViewSupport {
            DFS_TMP_SCHEMA,
            null,
            viewSQL,
   -        "SELECT * FROM TEST_SCHEMA.TEST_VIEW_NAME LIMIT 4",
   +        //"SELECT * FROM TEST_SCHEMA.TEST_VIEW_NAME LIMIT 4",
   +        viewSQL + " LIMIT 4",
            baselineColumns("customer_id", "order_id"),
            baselineRows(row(1.0,  1.0),
                         row(1.0,  2.0),
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] bozzzzo commented on pull request #2083: DRILL-7722: Add reproducer unit test

Posted by GitBox <gi...@apache.org>.
bozzzzo commented on pull request #2083:
URL: https://github.com/apache/drill/pull/2083#issuecomment-635129685


   The test fails in the same way as already reported in JIRA
   https://github.com/apache/drill/runs/715619115?check_suite_focus=true#step:6:10828


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org