[
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 6:48 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. It applies to
alias as well. 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.
But I think it is ok for now since column name should be the same for the
server side or the client side which should be taken from the submitted SQL.
was (Author: aliciashu):
[~jamestaylor] [~maryannxue] The temp schema is to work around Aggregate
projection to replace rowProjector. When we reach the point of projection,
should all the mergeSort already have occurred? 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.
> 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)
