Martin Peřina has uploaded a new change for review. Change subject: core: Replace log4j with slf4j in restapi ......................................................................
core: Replace log4j with slf4j in restapi Replaces log4j with slf4j in restapi module. Change-Id: I8109622c54eb2093753ab0c087d0d73d177aaa5a Bug-Url: https://bugzilla.redhat.com/1109871 Signed-off-by: Martin Perina <[email protected]> --- M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java M backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml 3 files changed, 4 insertions(+), 9 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/34179/1 diff --git a/backend/manager/modules/restapi/jaxrs/pom.xml b/backend/manager/modules/restapi/jaxrs/pom.xml index b3ef407..dcdf26d 100644 --- a/backend/manager/modules/restapi/jaxrs/pom.xml +++ b/backend/manager/modules/restapi/jaxrs/pom.xml @@ -71,11 +71,6 @@ </dependency> <dependency> - <groupId>log4j</groupId> - <artifactId>log4j</artifactId> - </dependency> - - <dependency> <groupId>org.easymock</groupId> <artifactId>easymockclassextension</artifactId> <version>${easymock.version}</version> diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java index 9ec8c77..3fcdd38 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java @@ -24,7 +24,6 @@ import javax.ws.rs.ApplicationPath; import javax.ws.rs.core.Application; -import org.apache.log4j.Logger; import org.ovirt.engine.api.common.invocation.Current; import org.ovirt.engine.api.restapi.logging.MessageBundle; import org.ovirt.engine.api.restapi.logging.Messages; @@ -66,11 +65,13 @@ import org.ovirt.engine.api.restapi.types.MappingLocator; import org.ovirt.engine.api.restapi.util.SessionHelper; import org.ovirt.engine.core.common.interfaces.BackendLocal; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; @ApplicationPath("/") public class BackendApplication extends Application { // The log: - private static final Logger logger = Logger.getLogger(BackendApplication.class); + private static final Logger log = LoggerFactory.getLogger(BackendApplication.class); // The messages bundle: private final MessageBundle messageBundle; @@ -110,7 +111,7 @@ backend = (BackendLocal) initial.lookup("java:global/engine/bll/Backend!org.ovirt.engine.core.common.interfaces.BackendLocal"); } catch (Exception exception) { - logger.error("Can't find reference to backend bean.", exception); + log.error("Can't find reference to backend bean.", exception); throw exception; } diff --git a/backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml b/backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml index 2f2fd3d..f086fe7 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml +++ b/backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml @@ -27,7 +27,6 @@ <module name="javax.ws.rs.api"/> <module name="javax.xml.bind.api"/> <module name="org.apache.commons.lang"/> - <module name="org.apache.log4j"/> <module name="org.codehaus.jackson.jackson-core-asl"/> <module name="org.jboss.resteasy.resteasy-jaxb-provider"/> <module name="org.jboss.resteasy.resteasy-jaxrs"/> -- To view, visit http://gerrit.ovirt.org/34179 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I8109622c54eb2093753ab0c087d0d73d177aaa5a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
