search

twitter / summingbird

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

Revert unneeded wrapper in DagOptimizer #584

Closed johnynek closed 9 years ago

johnynek commented 9 years ago

Improves #580

johnynek commented 9 years ago

Note, the new tests still pass, and we don't need to add the wrapped type in the dag optimizer (also we remove the castToKeyed).

egonina commented 9 years ago

+1, didn't see the castToPair option, thanks @johnynek

ianoc commented 9 years ago

One build is erroring due to some issues fetching packages by the looks. The other is green. Merging this as works locally for people.