[jira] [Commented] (CRUNCH-432) Adapt JoinStrategy API for Scrunch PTables

["Josh Wills (JIRA)" <ji...@apache.org>]

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

Josh Wills commented on CRUNCH-432:
-----------------------------------

Hey [~gabriel.reid], I was trying to commit this, but ran into a problem: the new Mapside Join API isn't available in 0.8, so I can only fix this in master as it is here. I get that we don't want to mess with the default behavior in 0.8, but I was hoping to correct the MapsideJoinStrategy argument ordering mistake inside of Scrunch-- or is that a bad idea and I should replicate the behavior in both places?

> Adapt JoinStrategy API for Scrunch PTables
> ------------------------------------------
>
>                 Key: CRUNCH-432
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-432
>             Project: Crunch
>          Issue Type: Bug
>          Components: Scrunch
>            Reporter: Josh Wills
>         Attachments: CRUNCH-432.patc, CRUNCH-432.patch, CRUNCH-432b.patch
>
>
> The JoinStrategy stuff in Java is one of my favorite Crunch APIs, and it would be great to make it easy to use them w/the Scrunch PTables as well.



--
This message was sent by Atlassian JIRA
(v6.2#6252)