Skip to content

Commit

Permalink
fix unexpected TimeSeriesLimitReached after HistogramLimitReached(#60752
Browse files Browse the repository at this point in the history
) (#61199)
  • Loading branch information
itn3000 authored Nov 5, 2021
1 parent d2415c4 commit 17fa0c8
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -302,7 +302,8 @@ private void RemoveInstrumentState(Instrument instrument, InstrumentState state)
{
lock (this)
{
return (!CheckTimeSeriesAllowed() || !CheckHistogramAllowed()) ?
// checking currentHistograms first because avoiding unexpected increment of TimeSeries count.
return (!CheckHistogramAllowed() || !CheckTimeSeriesAllowed()) ?
null :
new ExponentialHistogramAggregator(s_defaultHistogramConfig);
}
Expand Down

0 comments on commit 17fa0c8

Please sign in to comment.