Release/0.5.2rc2

Takes @egonina 's work and ads the final few bits to get green tests. Given its for the forth coming scalding release it seems good to just merge into develop with changes this much here.

@johnynek changes here are all around the scalding platform, could you take a look? Thanks!
I don't want to delete the code, as it may be useful. But clearly we need
tests.

We can ignore it for now.

Merge when green.


On Fri, Sep 5, 2014 at 1:12 PM, ianoc <notifications@github.com> wrote:

> In
> summingbird-scalding/src/main/scala/com/twitter/summingbird/scalding/store/DirectoryBatchedStore.scala:
>
> > @@ -105,7 +105,7 @@ class DirectoryBatchedStore[K <: Writable, V <: Writable](val rootPath: String)(
> >      val lastID = getLastBatchID(exclusiveUB, mode)
> >
> >      val src = WritableSequenceFile(rootPath + "/" + lastID.toString, 'key -> 'val)
> > -    val rdr = Reader { (fd: (FlowDef, Mode)) => TypedPipe.from(src.read(fd._1, fd._2), Fields.ALL) }
> > +    val rdr = Reader { (fd: (FlowDef, Mode)) => TypedPipe.from(src.read(fd._1, fd._2), Fields.ALL)(fd._1, fd._2, implicitly[TupleConverter[(K, V)]]) }
>
> Yeah that makes sense... we should scan for usage and kill if we can't
> find any?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twitter/summingbird/pull/545/files#r17203222>.
>



-- 
Oscar Boykin :: @posco :: http://twitter.com/posco