#536 Add MBean support for Log4j

Submitter:	PABLO OJANGUREN MENENDEZ
Branch:
Bugs:	536

Reviewers
Groups:
People:	vjrj
Repository:	kune

Description:

http://redmine.ourproject.org/issues/536 Add MBean support for Log4j

Added code to register default log4j MBean. All loggers are included in the MBean so all they are accessible.

Testing Done:

Compiled, executed server and checked via jconsole that MBean is available.

LGTM. Only a minor comment. You know, we use certain Eclipse code preferences for Kune project:
 dev-utils/kune.cleanup.eclipse.preferences.xml
 dev-utils/kune.formatter.eclipse.preferences.xml
so please re-format/clean-up these classes using these preferences before commiting. The diffs will be more accurate between commits.

Goooood job...

PABLO OJANGUREN MENENDEZ May 15, 2013, 11:59 a.m. (May 15, 2013, 11:59 a.m.)

I think I have already activated those preferences but I will check again. If your comment is regarding to the white space after the char '*' in comments, it's an eclipse's bug. I will remove them before committing changes.

Thank you!

Vicente J. Ruiz Jurado May 17, 2013, 2:36 p.m. (May 17, 2013, 2:36 p.m.)

I use AnyEditTools plugin to convert tabs to spaces and to remove trailing spaces (as commented in the DEV-GUIDE).

Also check that you have as "Save Actions" to "Format source code" so... when you save the files are formatted using our common preferences.

Anyway, commit! :-)

Vicente J. Ruiz Jurado June 13, 2013, 1:04 p.m. (June 13, 2013, 1:04 p.m.)
```
Commited by pablojan as r451cbc8.
```

status changed from pending to submitted

You have a pending review.

Other reviews

Your comment

Review Board 1.7.2

This change has been marked as submitted.