Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin,userportal: Improve the way logging works ...................................................................... webadmin,userportal: Improve the way logging works 1. all logging related code extracted into ApplicationLogManager 2. added "error is effectively the same as last one" detection: * on 1st occurence, error is logged as "Uncaught exception" * on 2nd occurence, error is logged as "Uncaught exception (2x)" * same message for any further occurences of the same error 3. error is remotely logged ONLY if both of following are true: * application (browser) window has focus * current error is NOT effectively the same as last one An error E1 is effectively the same as error E2 if and only if E1's stack trace has the same length as E2's stack trace and the stack trace elements are equal at any given index. Due to how GWT emulates java.lang.StackTraceElement, we're using StackTraceElement.toString() to capture all important info: declaringClass methodName fileName lineNumber Change-Id: I66c3c5c4ca2451403d1cdd8d29cca4823a90f4f2 Bug-Url: https://bugzilla.redhat.com/1358136 Signed-off-by: Vojtech Szocs <vszocs@redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseSystemModule.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/logging/ApplicationLogManager.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/ApplicationFocusManager.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/BaseApplicationInit.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/system/ApplicationInit.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/system/ApplicationInit.java 6 files changed, 150 insertions(+), 52 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61237 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66c3c5c4ca2451403d1cdd8d29cca4823a90f4f2 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: Alexander Wels <awels@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>