On 18/04/2007, at 2:46 PM, Scott Deboy wrote:

In DBAppender:
insertStatement.setLong(0, 0);
should be
insertStatement.setLong(1, 0);


done, well spotted.

Also, in CustomSQLDBReceiver:
Should
LoggingEvent event = new LoggingEvent(null,
eventLogger, timeStamp, levelImpl, message,
threadName,
throwableInfo,
ndc,
locationInfo,
properties);

be

LoggingEvent event = new LoggingEvent(eventLogger.getName(),
eventLogger, timeStamp, levelImpl, message,
threadName,
throwableInfo,
ndc,
locationInfo,
properties);
?


Done, that makes sense, not sure what I was smoking.

In dbreceiverjob:
Should we pass a logger name instead of null in loggingevent constructor (same as above)?


done.

Question: are we going to support event properties in the 1.2 stream? Being able to populate properties from MDC, throwableinfo and NDC are all pretty important...would be nice if all of the receivers could use this functionality (multicastappender/receiver, etc).


That's probably a next step.

Paul

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to