search

aphyr / tesser

Clojure reducers, but for parallel execution: locally and on distributed systems.
873 stars 39 forks source link

tesser.hadoop: reduce in two stages #4

Closed egamble closed 9 years ago

aphyr commented 9 years ago

This looks good to me, Evan. Couple minor questions and I think we'll be good to go. Are you getting good results in prod? If so, I'll cut straight to 1.0.1 and release for you. :)

egamble commented 9 years ago

Yes, good results in production. Added doc string, changed identity-mapper as you suggested (thought I had found that didn't work, but it does in fact work), etc.

aphyr commented 9 years ago

Oh, gotcha, the partitioner fun is conforming to another API! Yeah, looks good to me. Merging and releasing! :)