[ https://issues.apache.org/jira/browse/PHOENIX-1580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14391631#comment-14391631 ]
Alicia Ying Shu edited comment on PHOENIX-1580 at 4/2/15 1:15 AM: ------------------------------------------------------------------ [~maryannxue] After applying your patch, almost all of my tests failed with NPE. The NPE was from ParseNodeRewriter.java. So I added the following in ParseNodeRewriter.java. Does it look ok? if (from == null) return statement; Also many existing tests failed. Those all passed before your patch. Looks like ParameterCount gave us trouble again. Also got a lot of NULL result back. I have made all the changes asked by [~jamestaylor] was (Author: aliciashu): [~maryannxue] After applying your patch, almost all of my tests failed with NPE. The NPE was from ParseNodeRewriter.java. So I added the following in ParseNodeRewriter.java. Does it look ok? if (from == null) return statement; Also many existing tests failed. Those all passed before your patch. Looks like ParameterCount gave us trouble again. Also got a lot of NULL result back. Can you work on it if you think it is better later. I have clearly stated many database systems do not wrap UNION ALL with dummy select *. I have made all the changes asked by [~jamestaylor] > 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-v2.patch, Phoenix-1580-v3.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)