Zwu, this is not about your change, but log4j question.  What is suggested pattern ?

if(logger.isDebugEnabled())
       logger.debug()

or just
logger.debug()

Most of our code base uses the former and I can clean whenever I touch something of the former.