If you can push bijection out today, I can make the change by tomorrow. Let
me know if this works


On Tue, Feb 4, 2014 at 1:54 PM, P. Oscar Boykin <notifications@github.com>wrote:

> In
> storehaus-elasticsearch/src/main/scala/com/twitter/storehaus/elasticsearch/Injections.scala:
>
> > +import scala.util.Try
> > +import org.json4s._
> > +import native.JsonMethods._
> > +import com.twitter.bijection.Inversion.attempt
> > +import native.Serialization.{write, read}
> > +import com.twitter.algebird.Monoid
> > +import org.json4s.JValue
> > +import com.twitter.algebird.
> bijection.BijectedMonoid
> > +
> > +
> > +/**
> > + * Temporary home of Json algebra. Will be removed once Json4s bijections get merged in twitter-bijections
> > + * @author Mansur Ashraf
> > + * @since 1/8/14
> > + */
> > +object Injections {
>
> since the bijection code got merged, can we make this private so we don't
> get a binary incompatibility when we remove it?
>
> I want to publish this week. That or we can get bijection out today, and
> then fix this up tomorrow or thurs?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/twitter/storehaus/pull/205/files#r9435670>
> .
>