[
https://issues.apache.org/jira/browse/DRILL-6321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16481070#comment-16481070
]
ASF GitHub Bot commented on DRILL-6321:
---------------------------------------
amansinha100 commented on a change in pull request #1275: DRILL-6321: Lateral
Join and Unnest - star column prefix
URL: https://github.com/apache/drill/pull/1275#discussion_r189366569
##########
File path:
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/lateraljoin/TestLateralPlans.java
##########
@@ -90,8 +90,50 @@ public void testLateralSqlPlainCol() throws Exception {
@Test
public void testLateralSqlStar() throws Exception {
- String Sql = "select * from cp.`lateraljoin/nested-customer.json` t,
unnest(t.orders) t2 limit 1";
- test(Sql);
+ String Sql = "select * from cp.`lateraljoin/nested-customer.json` t,
unnest(t.orders) t2 limit 0";
+
+ testBuilder()
+ .unOrdered()
+ .sqlQuery(Sql)
+ .baselineColumns("c_name", "c_id", "c_phone", "orders",
"c_address", "o_id", "o_shop", "o_amount", "items")
Review comment:
I think this set of columns is not right. The UNNEST should produce only
the top level column since that's what the planner knows about.
De-constructing the underlying map in the data means that we will not be able
to specify a column alias for the unnest output: e.g UNNEST(t.orders) as
t2(o).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Lateral Join: Planning changes - enable submitting physical plan
> ----------------------------------------------------------------
>
> Key: DRILL-6321
> URL: https://issues.apache.org/jira/browse/DRILL-6321
> Project: Apache Drill
> Issue Type: Task
> Reporter: Parth Chandra
> Assignee: Chunhui Shi
> Priority: Major
> Labels: ready-to-commit
> Fix For: 1.14.0
>
>
> Implement changes to enable submitting a physical plan containing lateral and
> unnest.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
