split sla from critical time

SC used those headers so they still need to be set before sending to audit

Sent from my iPhone

> On 08 Aug 2014, at 01:48, John Simons <notifications@github.com> wrote:
> 
> In src/NServiceBus.Core/Monitoring/ProcessingStatisticsBehavior.cs:
> 
> > @@ -0,0 +1,24 @@
> > +﻿namespace NServiceBus
> > +{
> > +    using System;
> > +    using Pipeline;
> > +    using Pipeline.Contexts;
> > +
> > +    class ProcessingStatisticsBehavior : IBehavior<IncomingContext>
> > +    {
> > +        public void Invoke(IncomingContext context, Action next)
> > +        {
> > +            //since the audit captures the physical message headers then lets place them there
> According to the history of when these headers were introduced see here, it seems they were only introduced to support the critical time perf counter.
> Now because we have the "context" to store info we do not need to store this info in headers any more. So how about we remove them?
> It seems to be pointless to have 2 headers (start/stop), if we really need to expose this info in a header then one header "TotalTime" should suffice.
> 
> —
> Reply to this email directly or view it on GitHub.