Einav Cohen has posted comments on this change.
Change subject: core: provided cleared frontend logging
......................................................................
Patch Set 1: (1 inline comment)
- adding an engine-core maintainer to review changes in common entities.
- please note one inline comment. thanks.
....................................................
File
frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/GenericApiGWTServiceImpl.java
Line 190: StringBuilder builder = new StringBuilder(50);
Line 191: builder.append("Action type: "); //$NON-NLS-1$
Line 192: builder.append(actionType);
Line 193: builder.append(", Parameters: "); //$NON-NLS-1$
Line 194: builder.append(params);
missing "log.debug(builder.toString());"?
Line 195: }
Line 196: }
Line 197:
--
To view, visit http://gerrit.ovirt.org/12559
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I845aa47da03168f1a0f988203df7ab0a51e94804
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
