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

James Taylor commented on PHOENIX-1044:
---------------------------------------

+1. Nice job, [~maghamravikiran]. One minor nit: in your test you're only 
asserting that the record count is as expected, but you really should compare 
the value as well:
{code}
+        while (iterator.hasNext()) {
+            iterator.next();
+            recordsRead++;
+        }
+        assertEquals(1, recordsRead);
{code}

> Phoenix-Pig: No results returned unless all used columns are selected
> ---------------------------------------------------------------------
>
>                 Key: PHOENIX-1044
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1044
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.1
>            Reporter: Josh Mahonin
>            Priority: Minor
>         Attachments: 0001-PHOENIX-1044.patch
>
>
> Not critical, but a bit frustrating:
> I'm creating a PhoenixPigConfiguration via the following:
> > conf.setSelectColumns("FOO,BAR")
> > conf.setSelectStatement("SELECT FOO, BAR FROM MYTABLE WHERE BAZ=123")
> In SQuirrel, that query returns with results, but the going through Pig, it 
> returns nothing.
> I also get the same results if I try like so:
> > conf.setSelectColumns("FOO,BAR,BAZ")
> > conf.setSelectStatement("SELECT FOO, BAR FROM MYTABLE WHERE BAZ=123")
> The workaround is to modify the select statement to also include the 'BAZ' 
> column, even though I don't need it for computation:
> > conf.setSelectStatement("SELECT FOO, BAR, BAZ FROM MYTABLE WHERE BAZ=123")



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

Reply via email to