Hi Jonathan Kelly,
Thanks for the patch! A few comments:
1. I can't review the Puppet bits because I don't have enough familiarity with them. If you want to move the Puppet changes to a separate patch / JIRA, I can focus on reviewing the Spark update and RPM changes
2. I'm worried about compiling Spark with Ganglia, which is LPGL licensed. Spark seems to need this to be explicitly enabled, and I'm wondering if we need to do the same as well. (e.g., create an environmental variable or something and document it). I'll send an email to the dev list to get feedback on this.
3. The patch has a number of changes that seem purely cosmetic (e.g., changing the order of the source imports in the RPM SPEC file, re-organized lines in spark-core.install, etc.). These make the patch hard to read, and I don't want to accidentally introduce errors. Can you clean those up?