You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Jesse Yates (JIRA)" <ji...@apache.org> on 2016/02/10 21:49:18 UTC

[jira] [Updated] (PHOENIX-2674) PhoenixMapReduceUtil#setInput doesn't honor condition clause

     [ https://issues.apache.org/jira/browse/PHOENIX-2674?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jesse Yates updated PHOENIX-2674:
---------------------------------
    Attachment: phoenix-2674-v0-without-test.patch

Attaching a simple patch that just builds the select statement. Obviously, people could just do this themselves, but then we should remove this method (not easy to do outside of major version boundaries) or fix it kinda sorta like this.

> PhoenixMapReduceUtil#setInput doesn't honor condition clause
> ------------------------------------------------------------
>
>                 Key: PHOENIX-2674
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2674
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Jesse Yates
>         Attachments: phoenix-2674-v0-without-test.patch
>
>
> The parameter is completely unused in the method. Further, it looks like we don't actually test this method or any m/r tools directly.
> It would be good to (a) have explicit tests for the MapReduce code - rather than relying on indirect tests like the index util - and, (b) have an example in code for using the mapreduce tools, rather than just the web docs (which can become out of date).



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