Option to select the fields for output in templatesource

Option to select the fields for output in templatesource
Good point. Forgot about it.

We really need some kind of better versioning handling with CI. It would be
great to always have two dev branches: one that is binary compatible with
the previous release and one that had that plus more. It would be nice to
publish the binary compatible stuff on a faster cadence and wire the
checking of mima into the CI.

On Tue, Nov 25, 2014 at 8:12 AM, ianoc <notifications@github.com> wrote:

> In scalding-core/src/main/scala/com/twitter/scalding/TemplateSource.scala:
>
> >   */
> >  case class TemplatedTsv(
> >    override val basePath: String,
> >    override val template: String,
> >    override val pathFields: Fields = Fields.ALL,
> >    override val writeHeader: Boolean = false,
> > -  override val sinkMode: SinkMode = SinkMode.REPLACE)
> > +  override val sinkMode: SinkMode = SinkMode.REPLACE,
> > +  override val fields: Fields = Fields.ALL)
>
> I think with the changes recently on the JDBC source we've already lost
> binary compatibility. Not sure if you think its worth the extra code bloat
> to keep it for this class? Maybe?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twitter/scalding/pull/1061/files#r20882272>.
>



-- 
Oscar Boykin :: @posco :: http://twitter.com/posco