Liron Ar has posted comments on this change.

Change subject: core: log using INFO logging level rather than ERROR
......................................................................


Patch Set 1:

@Michael - 
"No reason for stack trace, no reason for info message on cancel event, it 
should be debug. There are can be 1000 of cancelled events every second, no 
reason for info message" -

as the event queue is rather new mechanism, at least for now i prefer to keep 
the stack trace - as the issues are not always easy to reproduce.
at the moment i prefer to lower the severity and keep the stacktrace, the next 
step would be to change it to DEBUG level rather then provide another temporary 
solution, i think that for now that's satisfying solution.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: If192b3ccc5f5310de4fd80ca7fb4a7d74d52e1df
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to