There already existed an o.a.l.log4j.core.appender.db package that was empty. I 
discovered that I could create a base AbstractDatabaseAppender that took care 
of some operations common to JDBC, JPA /and/ NoSQL appenders. That led me to 
use the existing db package as follows:

db/
    common DB appender classes
    jdbc/
        JDBC appender classes
    jpa/
        JPA appender classes
    nosql/
        NoSQL appender classes

I think that package structure makes a lot of sense. I'm open to convincing 
arguments otherwise.

Nick

On Apr 27, 2013, at 9:37 AM, Gary Gregory wrote:

> I think we should keep the packaging simpler: o.a.l.log4j.core.appender.jpa
> 
> Gary
> 
> On Apr 27, 2013, at 10:15, Nick Williams <nicho...@nicholaswilliams.net> 
> wrote:
> 
>> o.a.l.log4j.core.appender.db.jpa
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: log4j-dev-unsubscr...@logging.apache.org
> For additional commands, e-mail: log4j-dev-h...@logging.apache.org
> 

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to