twitter / summingbird

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

Get rid of `WireFmt` and `RuntimeContext` in `OperationContainer` #697

Closed ttim closed 7 years ago

ttim commented 7 years ago

In this PR I removed WireFmt and RuntimeContext type parameters from OperationContainer.

Main reason to do that was to make OperationContainer composable and responsible only for how to do computations on inputs. Therefore WireFmt moved to places which wraps OperationContainer in Storm.

RuntimeContext wasn't really used anywhere and I removed it for simplicity.