search

wr0ngway / log4r-gelf

A Log4r appender for logging to a gelf sink, e.g. the graylog2 server
MIT License
17 stars 5 forks source link

Non synched notify #6

Closed davebenvenuti closed 10 years ago

davebenvenuti commented 10 years ago

gelf 1.4 and up is threadsafe, so there's really no need to synch the notify call. Removing the synch block makes a pretty substantial performance difference in our case

wr0ngway commented 10 years ago

Sorry, missed this in my inbox, looks good