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

[GitBox <gi...@apache.org>]

nwangtw opened a new pull request #2993: Nwang/make ispout compatible to storm 1.x
URL: https://github.com/apache/incubator-heron/pull/2993
 
 
   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).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services