twitter / scalding

A Scala API for Cascading
http://twitter.com/scalding
Apache License 2.0
3.49k stars 703 forks source link

fix JobTest to use local mode #1845

Closed fwbrasil closed 6 years ago

fwbrasil commented 6 years ago

PR #1792 changed the way the configuration is created for JobTest. Instead of instantiating a JobConf it instantiates Configuration, which doesn't include a few configuration files. One of these files is hadoop-mapreduce-client-core-2.6.0.t01.jar!/mapred-default.xml, that sets mapreduce.framework.name as local by default.

cc/ @dieu @ttim @pankajroark @johnynek

fwbrasil commented 6 years ago

@johnynek @dieu thank you for the quick review! Could you merge it?