Closed tuterbatuhan closed 5 years ago
Hey @tuterbatuhan,
looks really good. I added some minor comments. I'm not sure if we should also put the storm native aggregation functions in the connector. But they are just used in the demo right? I think then its ok.
If you fixed the comments. I will merge the pull request.
Best, Philipp
A connector implementation for Apache Storm including window function implementation and a working demo