twitter / scalding

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

Make TypedPipe abstract class for better binary compatibility #1769

Closed johnynek closed 6 years ago

johnynek commented 6 years ago

in 2.11, every new method on a trait breaks binary compatibility. Before we release 0.18, we should make this change.

johnynek commented 6 years ago

@piyushnarang what do you think?

or @fwbrasil @ianoc

ianoc commented 6 years ago

Can't think of a good reason off hand this isn't a strictly better setup

lgtm

piyushnarang commented 6 years ago

Yeah I think it sounds good to me as well. 👍