You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/07/22 17:41:08 UTC

[GitHub] [spark] nkarpov opened a new pull request #25232: [SPARK-28475][CORE] Add regex MetricFilter to GraphiteSink

nkarpov opened a new pull request #25232: [SPARK-28475][CORE] Add regex MetricFilter to GraphiteSink
URL: https://github.com/apache/spark/pull/25232
 
 
   ## What changes were proposed in this pull request?
   
   Today all registered metric sources are reported to GraphiteSink with no filtering mechanism, although the codahale project does support it.
   
   GraphiteReporter (ScheduledReporter) from the codahale project requires you implement and supply the MetricFilter interface (there is only a single implementation by default in the codahale project, MetricFilter.ALL).
   
   Propose to add an additional regex config to match and filter metrics to the GraphiteSink
   
   ## How was this patch tested?
   
   Small change, manually tested.
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org