serilog-contrib / serilog-sinks-splunk

A Serilog sink that writes to Splunk
https://splunk.com
Apache License 2.0
46 stars 47 forks source link

BoundedConcurrentQueue does not accept a negative queue limit. Using … #138

Closed JuanjoFuchs closed 4 years ago

JuanjoFuchs commented 4 years ago

I made a mistake, BoundedConcurrentQueue does not accept a negative queue limit. We need to use the default specified in PeriodicBatchingSinkOptions.QueueLimit

PeriodicBatchingSinkOptions.cs#L45

JuanjoFuchs commented 4 years ago

I've noticed 100,000 is also the default used by ElasticSearchSink which also depends on the PeriodicBatchingSink