search

kite-sdk / kite

Kite SDK
http://kitesdk.org/docs/current/
Apache License 2.0
394 stars 263 forks source link

CDK-976: Fix task configuration in MR #371

Closed rdblue closed 9 years ago

rdblue commented 9 years ago

This sets the default configuration using the one passed to the InputFormat or OutputFormat when running in a MR task. To avoid this setting leaking into tests, this adds DefaultConfiguration.init(Configuration), which will set the default conf only if it has not already been set via set or init. (Otherwise, test fail.)

This replaces #368.

rdblue commented 9 years ago

@tomwhite, could you take a look at this? I think it is correct to set the default config, but I needed to avoid setting it when not running in MR.

rdblue commented 9 years ago

Fixed, thanks Tom!