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

Gabriel Reid commented on CRUNCH-432:
-------------------------------------

+1

Although I have to admit I don't know anything about Scala (although this gives 
me even more motivation to start learning it).

I was wondering how you would have made a version that would work correctly 
with 0.8 and master, but now I get it. This will also be nice and easy to 
change if/when we totally get rid of the legacy MapsideJoin. I also like the 
addition of the SwapFn -- that's something I always end up writing myself for 
other stuff, and I've been meaning to add it for a long time.

> 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, 
> CRUNCH-432c.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)

Reply via email to