lukess edited a comment on issue #9848: [FLINK-14332] [flink-metrics-signalfx] 
add flink-metrics-signalfx
URL: https://github.com/apache/flink/pull/9848#issuecomment-542908129
 
 
   @zentol thanks for comment, let me try to answer questions:
   1. Are you talking about something like bahir-flink (connector) for metrics? 
A central place to release flink metrics extensions I think is good. I can move 
this plugin (and others) as well. My current issue is that our org heavily rely 
on SFX alert. I do that for our production and for others who using SFX metrics.
   2. I think to bump dropwizard probably cannot solve the 
io.dropwizard.metrics:metrics-core out-of-sync problem when signalfx-codahale 
and signalfx-java bump the io.dropwizard.metrics:metrics-core dependency. For 
ScheduledDropwizardReporter users do not need SFX libs as well.
   3. I think this is a good way to go. Let me check the new JMXReporter.
   
   updated:
   is MetricReporter.open(MetricConfig config) planning to deprecate? From 
JMXReporterFactory looks like the config will be parsed and then new 
JMXReporter(args). If this is a goal I think I gonna to skip parse config again 
during MetricReporter.open life cycle.

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


With regards,
Apache Git Services

Reply via email to