Hi jon ! You DEFINETLY do NOT want to put in that change to mapreduce_shuffle.
The reason is that for 0.8.0 (head) release, the parameter is correct.
You are deploying 0.7.0, where you manually change that parameter, but we dont want to bake that change into the puppet recipes.
Makes sense? otherwise, I like adding yarn and mapred components.
Please revert the change to the xml file, and maybe replace it with a comment specifying that for 0.7.0, this should be mapreduce.shuffle, for 0.8+, mapreduce_shuffle .