[ 
https://issues.apache.org/jira/browse/PHOENIX-1580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14390020#comment-14390020
 ] 

Alicia Ying Shu edited comment on PHOENIX-1580 at 4/1/15 7:28 AM:
------------------------------------------------------------------

[~jamestaylor] [~maryannxue]  The temp schema is to work around Aggregate 
projection to replace rowProjector. OrderBy expression is normally column name, 
it should be the same no matter in the server or in the client. The temp schema 
is already built with the projection names (column names) as specified in the 
SQL. All UnionAllIT tests passed with Plan compiled Order By and Limit. 

Wrapping another Select over current Union All selects we need to specify a 
table. A dummy table or Null table did not work since parser needs a real table 
to resolve column information etc. I have to say if we have taken my very early 
approach of passing down rowProjector, we can simplify the approach quite bit. 
Given that column names are used to construct the temp schema, no need to wrap 
another select over. 



was (Author: aliciashu):
[~jamestaylor] [~maryannxue]  The temp schema is to work around Aggregate 
projection to replace rowProjector. OrderBy expression is normally column name, 
it should be the same no matter in the server or in the client. It applies to 
alias as well. The temp schema is already built with the projection names 
(column names) as specified in the SQL. I did not see an issue here. All 
UnionAllIT tests passed with Plan compiled Order By and Limit. 

Wrapping another Select over current Union All selects we need to specify a 
table. A dummy table or Null table did not work since parser needs a real table 
to resolve column information etc. I have to say if we have taken my very early 
approach of passing down rowProjector, we can simplify the approach quite bit. 
Given that column names are used to construct the temp schema, no need to wrap 
another select over. 


> Support UNION ALL
> -----------------
>
>                 Key: PHOENIX-1580
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1580
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Alicia Ying Shu
>            Assignee: Alicia Ying Shu
>         Attachments: PHOENIX-1580-grammar.patch, Phoenix-1580-v1.patch, 
> phoenix-1580-v1-wipe.patch, phoenix-1580.patch, unionall-wipe.patch
>
>
> Select * from T1
> UNION ALL
> Select * from T2



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to