Juan Hernandez has posted comments on this change.

Change subject: core: Update the version of slf4j to 1.7.2
......................................................................


Patch Set 1:

Roy, using the slf4j API instead of the log4j API shouldn't be a problem, even 
without this patch. This just updates the version so that we use the same 
provided the application server. In addition this version also supports vararg 
arguments which simplifies its use. Previous one supported only up to 2 
arguments.

I think that slf4j is a good alternative to org.ovirt.engine.core.utils.log.Log 
in general.

-- 
To view, visit http://gerrit.ovirt.org/16751
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I64eaaf8e499ee66049c24aad8189b406a3948491
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to