Martin Peřina has posted comments on this change. Change subject: tools: Replaces log4j with JUL as backend for notifier ......................................................................
Patch Set 1: Unfortunately there's some bug in java.util.logging classloader which is unable to load custom formatter when application is executed using jboss modules. JUL implementation of formatter initialization is very bad, exception raised during formatter classloading is dropped [1] So we have following options: 1) Don't use our custom formatter -> it would mean that in tools we will have different log level names than in engine (FINE instead of DEBUG, WARNING instead of WARN, SEVERE instead of ERROR and FATAL) 2) Initialize logging using API in code and don't rely in properties initialization -> more code [1] http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7u40-b43/java/util/logging/LogManager.java?av=f#1318 -- To view, visit http://gerrit.ovirt.org/33774 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I10216f8530e23c43942cdf28fe2cb306d7e6df71 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Martin Peřina <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
