Re: [log4j] MongoDB Appender changes heads up.

2018-01-10 Thread Gary Gregory
The build should run as is. If you want to run the MongoDB tests, then build with Java 8. Otherwise, these tests will be skipped. Previously, the MongoDB tests were all annotated with @Ignore, so we are better off now IMO. Gary On Wed, Jan 10, 2018 at 10:04 PM, Ralph Goers wrote: > So now we ne

Re: [log4j] MongoDB Appender changes heads up.

2018-01-10 Thread Ralph Goers
So now we need Java 7, 8 and 9 to perform a build? Did you update the build documentation? Is that the only tool available to test MongoDB? Ralph > On Jan 10, 2018, at 8:18 PM, Gary Gregory wrote: > > The changes are both binary compatible and behavior compatible. The new > code kicks in only

Re: [log4j] MongoDB Appender changes heads up.

2018-01-10 Thread Gary Gregory
The changes are both binary compatible and behavior compatible. The new code kicks in only if the NoSqlAppender node contains a MessageLayout child, which was not possible before. I also added some code to test against an embedded MongoDB instance by using code from de.flapdoodle.embed.mongo. This

Re: [log4j] MongoDB Appender changes heads up.

2018-01-10 Thread Remko Popma
No objection, but please do this in a way that existing users of this appender are not affected. So if the new behavior is incompatible with the current behavior, then the new behavior should be switched on explicitly in configuration. Remko (Shameless plug) Every java main() method deserve

[log4j] MongoDB Appender changes heads up.

2018-01-10 Thread Gary Gregory
Hi All: I need for the MongoDB Appender to behave similarly to the JMS Appender regarding MapMessages. What I plan on doing: Like with the JMS Appender, if the MongoDB element contains a child MessageLayout element, the MongoDB appender will apply this layout. Like with the JMS Appender, my goa