marcelcaraciolo/recsys-mapreduce-mrjob — pull request #1 by irskep (2012-08-24 17:41:13+00:00)
Title: Various improvements to docs and mrjob 0.4 compatibility

Hi! I'm the maintainer of mrjob. I really enjoyed your blog post (it's now linked from the mrjob readme). I just wanted to fix a couple of things to make sure your example continues to work in the future and is easy to get running.

Stuff I did:

* Readme explains usage a little better and uses paths relative to the project root
* Move data files into `data/`
* Instead of raw print statement for last reducer, use a custom protocol (you shouldn't use print statements in a job)
* Remove an unused import
* Use keyword args to `mr()` instead of positional args, which have been deprecated for a while
* Link to blog post from readme