The 'prepare' operation considered harmful in #Algebird aggregation
http://erikerlandson.github.io/blog/2015/11/24/the-prepare-operation-considered-harmful-in-algebird/ …
@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... :DThanks. Twitter will use this to make your timeline better. UndoUndo
-
-
-
@manyangled I don't think we need the monoid constraint in that interface; you might need it for certain usage patterns.Thanks. Twitter will use this to make your timeline better. UndoUndo
-
-
-
@posco@avibryant https://github.com/twitter/algebird/pull/501 … (and you were right, I based my solution on the existing append method)
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.