search

twitter / summingbird

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

Remove catch Throwable, replace with NonFatal #641

Closed johnynek closed 8 years ago

johnynek commented 8 years ago

It's possible this is hiding OOM errors in some cases. We really shouldn't catch OOM or linkage errors.

benpence commented 8 years ago

LGTM