FoundatioFx / Foundatio

Pluggable foundation blocks for building distributed apps.
Apache License 2.0
1.96k stars 243 forks source link

Replace Metrics.Net with AppMetrics #114

Open edwardmeng opened 6 years ago

edwardmeng commented 6 years ago

The library Metrics.Net is only available for netfx and only can be deployed on windows servers. But the Foundatio currently supports netstandard 2.0, the Metrics.Net should be replaced with AppMetrics which supports netstandard, so that we can deployed on linux or unix.

ejsmith commented 6 years ago

@edwardmeng I agree. I have been wanting to make some changes to the Foundatio metrics contract to allow adding tags/dimensions to metrics and I also wanted to create an AppMetrics implementation as well.

niemyjski commented 6 years ago

They are working on a metrics.net net standard implementation and it was user contribed (on a side note). I also agree with the both of you but against removing metrics.net :)

ejsmith commented 6 years ago

Wouldn't remove it. Would just add another implementation.

edwardmeng commented 6 years ago

@ejsmith I am wondering how to support tags/dimensions for statsd implementation. As I know the statsd does not support it be default.

ejsmith commented 6 years ago

Sorry for the late reply @edwardmeng. Yeah, there are some implementations that would not be able to to support tagging/dimensions. So those would just be ignored. I am envisioning that we just add a method to let you create scopes that contain the dimensions on them and they just return the normal IMetricsClient back so none of the core contract would change. It would just be a matter of adding a method like this:

public interface IMetricsClient : IDisposable {
   IMetricsClient BeginScope(IDictionary<string, object> data);
   void Counter(string name, int value = 1);
   void Gauge(string name, double value);
   void Timer(string name, int milliseconds);
}

Then we would have extension methods with a fluent builder that let you build up the scope dictionary using a fluent lambda syntax which would be used something like this:

using (var scoped = client.StartScope(s => s.Tag("mytag").Dimension("host", "myhost1"))) {
  scoped.Counter("mycounter");
}

Tags would just be added to the dictionary without any values. We could also add extension methods for the Counter, Gauge, Timer methods that let you add scope data to individual metrics by just wrapping the method in a BeginScope.