nathanmarz / cascalog

Data processing on Hadoop without the hassle.
Other
1.38k stars 178 forks source link

add config setting to retain outputcollector. #160

Closed fs111 closed 11 years ago

fs111 commented 11 years ago

This is needed for Cascading 2.2 compatibility and fixes/works around https://github.com/nathanmarz/cascalog/issues/159

fs111 commented 11 years ago

Code changed following your suggestions. Using the constant will however break, if you use older versions of cascading. I don't know, if that is a good idea.

sritchie commented 11 years ago

Oh, that's actually a really good point :) Duh. We don't want to break backwards compatibility yet. Would you mind switching the setting back? I'll merge right away after that. Sorry for the hassle.

fs111 commented 11 years ago

Done.

sritchie commented 11 years ago

Thanks!