The 'prepare' operation considered harmful in #Algebird aggregation
http://erikerlandson.github.io/blog/2015/11/24/the-prepare-operation-considered-harmful-in-algebird/ …
@manyangled it feels wrong to me to extend Monoid to know about a whole new type. Aggregator already knows about e:A, better there.
-
-
@avibryant I can see the drawbacks. At the end of the day my priority is mostly taking advantage of the faster formulation, in some way -
@manyangled well, I'm happy to merge a PR that just adds update(b: B, a: A) to the Aggregator trait with a default implementation. -
@avibryant I'll see what I can do... :D
End of conversation
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.