[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2664 I think if we can shade all the storm-druid dependencies we could probably leave it there. At-least a few of the other storm connectors have not been updated in a while but we still keep

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2664 Got a response https://lists.apache.org/thread.html/22fa072edfdcb5a2fdb06e8c99daf8b2635da78e87c28e72b332ac0e@%3Cdev.druid.apache.org%3E. Sounds like Kafka is the recommended way to ingest into Druid,

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2664 Lets wait for a response. I think we can add some notes saying that storm-druid is deprecated and shade its dependencies than completely removing it since there are some users using it. This

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2664 I sent them an email here https://lists.apache.org/thread.html/4114eb25ff1e570927aa283c4127f8c83fb3bbee631ddebeee729f4c@%3Cdev.druid.apache.org%3E. I think if it turns out that Tranquility is

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2664 @revans2 thanks for understanding the concerns and agree that we should do something about the long term support for tranquility. We can probably poke them like @srdo suggested to get out a

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2664 @arunmahadevan and @srdo https://issues.apache.org/jira/browse/STORM-2882 is for adding back in shading to the storm client. I am happy to take that up next. My real concern is the long

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-08 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2664 I think shading Curator in storm-druid seems like the best option, and maybe poking a bit at the Druid guys to get a release of Tranquility out. If it turns out that they are dropping Tranquility

[GitHub] storm issue #2664: STORM-2884: Remove storm-druid

2018-05-07 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2664 @revans2 , I don't think we should completely drop the plugin since there are users using it. There were a few options proposed https://github.com/apache/storm/pull/2498#issuecomment-355423380