split sla from critical time

Agreed

Sent from my iPhone

> On 08 Aug 2014, at 01:51, John Simons <notifications@github.com> wrote:
> 
> In src/NServiceBus.Core/Monitoring/SLA/SLAMonitoringConfig.cs:
> 
> >      {
> >          /// <summary>
> >          /// Enables the NServiceBus specific performance counters with a specific EndpointSLA.
> >          /// </summary>
> > -        public static ConfigurationBuilder EnablePerformanceCounters(this ConfigurationBuilder config, TimeSpan sla)
> > +        public static ConfigurationBuilder EnableSLACounter(this ConfigurationBuilder config, TimeSpan sla)
> Regarding the naming of the extension methods. I would go one step further and include the word "Performance".
> So I would rename this one to be EnableSLAPerfomanceCounter.
> My reasoning behind it is if we just call it "Counter", where is this counter data captured?
> 
> —
> Reply to this email directly or view it on GitHub.