Re: Bug in MetricConsumer

2017-01-15 Thread Jungtaek Lim
Just filed https://issues.apache.org/jira/browse/STORM-2293 and submitted a patch for master, 1.x, 1.0.x branches. I manually tested this so I think the patch will fix your problem, but I'd be still appreciated when you can verify my patch. - Jungtaek Lim (HeartSaVioR) 2017년 1월 13일 (금) 오후 6:44,

Re: Bug in MetricConsumer

2017-01-13 Thread Jungtaek Lim
Cool :) Btw, I've craft a patch for this, and will submit a pull request after you filed to issue. It should be great if you could help me verifying the patch if you really don't mind. Thanks, Jungtaek Lim (HeartSaVioR) 2017년 1월 13일 (금) 오후 5:33, Adrien Carreira 님이 작성:

Re: Bug in MetricConsumer

2017-01-13 Thread Adrien Carreira
I've set up storm.local.hostname on storm.yaml that is used for nimbus and supervisor. I've just removed this one, restarted nimbus and all supervisor and it's working :D I will submit a bug :-) 2017-01-12 17:54 GMT+01:00 Jungtaek Lim : > Did you set up storm.local.hostname

Re: Bug in MetricConsumer

2017-01-12 Thread Jungtaek Lim
Did you set up storm.local.hostname only for storm clusters, or also the node you're submitting jar from? Anyway I think this is not intended behavior (a bug). Could you file an issue to JIRA? https://issues.apache.org/jira/browse/STORM Please let me know if you want me to file this instead.

Re: Bug in MetricConsumer

2017-01-12 Thread Adrien Carreira
You are right, My topology config contains : storm.local.hostname "crwl-agent1" But why ? I haven't set up this value, just in each storm.yaml, because either I see localhost insteed of the hostname... 2017-01-12 10:26 GMT+01:00 Jungtaek Lim : > Hi Adrien, > > could you

Re: Bug in MetricConsumer

2017-01-12 Thread Jungtaek Lim
Hi Adrien, could you check topology config has 'storm.local.hostname'? srcWorkerHost uses the value of 'storm.local.hostname' if available. It's intended to read that value for each node but there might be edge case like reading topology config instead of node's config value. Thanks, Jungtaek

Re: Bug in MetricConsumer

2017-01-12 Thread Adrien Carreira
Nope :/ 2017-01-11 18:28 GMT+01:00 Jacob Johansen : > > > Do you have the hostname on all the supervisors machines named the same? > > On January 11, 2017 at 10:30:03 AM, Adrien Carreira (a...@reportlinker.com) > wrote: > > Hi Guys, > > I'm running a topology on 3

Re: Bug in MetricConsumer

2017-01-11 Thread Jacob Johansen
Do you have the hostname on all the supervisors machines named the same? On January 11, 2017 at 10:30:03 AM, Adrien Carreira (a...@reportlinker.com) wrote: Hi Guys, I'm running a topology on 3 supervisor. I've registred a custom MetricsConsumer class that output metrics on Elasticsearch. My