search

twitter / summingbird

Streaming MapReduce with Scalding and Storm
https://twitter.com/summingbird
Apache License 2.0
2.14k stars 267 forks source link

Audit and remove all uses or plus in favor of sumOption #304

Open johnynek opened 11 years ago

johnynek commented 11 years ago

Semigroup/Monoid.plus works one at a time, sumOption can do big optimizations for sequences. Strongly prefer to use sumOption when adding a sequence.

jnievelt commented 11 years ago

Probably obvious, but I just wanted to make a note that the merge store read would be a great place to have this.