[GitHub] [incubator-heron] nwangtw opened pull request #2993: Nwang/make ispout compatible to storm 1.x

["nwangtw (GitHub)" <gi...@apache.org>]

Currently in storm-compatibility API, the spout and bolt interfaces and derived classes are following storm 2.0 API. The config parameter for open() and prepare() functions has type Map<String, Object>.

Heron was designed to be compatible with Storm in API layer. However, when heron was developed a  few years ago, storm was pre 1.0 (2016 April). Storm 1.x API didn't change much in API layer, but 2.0 is not compatible although the changes are small. In conclusion, it might be better to freeze the storm compatibility layer to be compatible with storm 1.x so that we don't need to change the layer time by time and breaks existing topologies. (Even if we want to be compatible with storm 2.0, it is better to support both until the heron core code is really complete.)

Furthermore, storm 2.0 is not released yet so far so the API might change in future (although not likely).

[ Full content available at: https://github.com/apache/incubator-heron/pull/2993 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org

[GitHub] [incubator-heron] huijunwu closed pull request #2993: Nwang/make spout/bolt compatible to storm 1.x

["huijunwu (GitHub)" <gi...@apache.org>]

[ pull request closed by huijunwu ]

[ Full content available at: https://github.com/apache/incubator-heron/pull/2993 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org