NLoggerFactory uses thread unsafe Dictionary

microsoftarchive / data-pipeline

Exploring the problem of high-scale data ingestion on Azure

MIT License

6 stars 4 forks source link

NLoggerFactory uses thread unsafe Dictionary #45

Open stascode opened 9 years ago

stascode commented 9 years ago

NLoggerFactory.Create(string logName) uses thread unsafe dictionary and does not have thread locks around it. Solution would be to move to ConcurrentDictionary

bennage commented 9 years ago

I'd actually like to move away from NLog entirely.

bennage commented 9 years ago

Can you create a pull request with this change?