I chose do use an EvaluatorCollection because order does not matter. With a Filter the order is obviously important and a linked list is the ideal solution. When an evaluator causes a flush the buffer is sent and the remaining evaluators do not get a chance to process the LoggingEvent.
However, a linked list would work just as well and would be consistent with the implementation of filters. What is the best solution? -----Original Message----- From: Ron Grabowski (JIRA) [mailto:[EMAIL PROTECTED] Sent: Wed 12/20/2006 11:15 PM To: log4net-dev@logging.apache.org Subject: [jira] Commented: (LOG4NET-108) [PATCH] add support for multiple evaluators in BufferingAppenderSkeletan [ http://issues.apache.org/jira/browse/LOG4NET-108?page=comments#action_12460137 ] Ron Grabowski commented on LOG4NET-108: --------------------------------------- Why did you choose to use an EvaluatorCollection instead of a linked list similiar to how IFilters are processed on the AppenderSkeleton? If the EvaluatorCollection has 5 items in it and the 1st item causes a flush do the remaining 4 items still get a chance to process the LoggingEvent? > [PATCH] add support for multiple evaluators in BufferingAppenderSkeletan > ------------------------------------------------------------------------ > > Key: LOG4NET-108 > URL: http://issues.apache.org/jira/browse/LOG4NET-108 > Project: Log4net > Issue Type: Improvement > Components: Appenders, Core > Reporter: Drew Schaeffer > Priority: Minor > Attachments: EvaluatorCollection.cs, patch-file.diff > > > Currently BufferedAppenders only have support for one evaluator and one lossy > evaluator. This is fine when the only available evaluator is LevelEvaluator > (as multriple LevelEvaluators do not make sense) but with the addition of > ExceptionEvaluator (LOG4NET-107) it would be nice for > BufferingAppenderSkeletan to have a collection of evaluators. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
<<winmail.dat>>
