arvindshmicrosoft commented on a change in pull request #270: Support Accumulo 
installs on Microsoft Azure
URL: https://github.com/apache/fluo-muchos/pull/270#discussion_r312262916
 
 

 ##########
 File path: ansible/roles/accumulo/templates/hadoop-metrics2-accumulo.properties
 ##########
 @@ -20,3 +20,9 @@ 
accumulo.sink.graphite.class=org.apache.hadoop.metrics2.sink.GraphiteSink
 accumulo.sink.graphite.server_host={{ groups['metrics'][0] }}
 accumulo.sink.graphite.server_port=2004
 accumulo.sink.graphite.metrics_prefix=accumulo
+
+*.sink.statsd.class=org.apache.hadoop.metrics2.sink.StatsDSink
 
 Review comment:
   Thanks for the question, Keith. The [Azure Log 
Analytics](https://docs.microsoft.com/en-us/azure/azure-monitor/platform/data-sources-collectd)
 agent can send collectd metrics to [Azure Monitor 
Logs](https://docs.microsoft.com/en-us/azure/azure-monitor/). Given there is 
Hadoop metrics2 support for a statsd sink, and given there is a statsd plugin 
for collectd, it allows us to send metrics from Accumulo, Hadoop and Zookeeper 
through to Azure Monitor. 
   
   This change is therefore specifically to support our changes for 
cluster_type == azure. If you prefer that we make this conditional, that's 
definitely something we can do.

----------------------------------------------------------------
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