Moti Asayag has posted comments on this change.
Change subject: core: add isTraceEnabled to Log class
......................................................................
Patch Set 2: No score
(2 inline comments)
....................................................
File
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/log/Log.java
Line 83: }
Line 84:
Line 85: @Override
Line 86: public void trace(Object arg0) {
Line 87: if(log.isTraceEnabled()) {
please run the formatter for the code - there should be a space after the if
Line 88: log.trace(addPrefixToLogMessage(arg0));
Line 89: }
Line 90: }
Line 91:
Line 90: }
Line 91:
Line 92: @Override
Line 93: public void trace(Object arg0, Throwable arg1) {
Line 94: if(log.isTraceEnabled()) {
same
Line 95: log.trace(addPrefixToLogMessage(arg0), arg1);
Line 96: }
Line 97: }
Line 98:
--
To view, visit http://gerrit.ovirt.org/10873
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1e55a6964ed317dbe10bf3bd28af8b187edb90e3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
