[LOG4NET-32] AdoNetAppender losing first entry

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2.9
• Fix Version/s: 1.2.10
• Component/s: Appenders
• Environment: VB.Net 2003, .Net 1.1.4322 SP1, SQL Server 2000, Windows XP Pro
• Assignee: Nicko Cadell
• Reporter: Richard Gee
• Created: 03/Jun/05 17:32
• Updated: 09/Jun/05 02:15
• Resolved: 09/Jun/05 02:15

Description

AdoNetAppender is losing the first log entry, caused by the entry being discarded as the underlying buffer cycles for the first time.

Probable error at line 400 in BufferingAppenderSkeleton.cs:

if (discardedLoggingEvent != null && m_lossyEvaluator != null && m_lossyEvaluator.IsTriggeringEvent(discardedLoggingEvent))

If you don't give a lossy evaluator the event is discarded, regardless of whether the buffer is lossy or not. This line should also check if the buffer is lossy, and send the discarded event if it isn't:

if (discardedLoggingEvent != null && (!m_lossy || (m_lossyEvaluator != null && m_lossyEvaluator.IsTriggeringEvent(discardedLoggingEvent))))