Michael Pasternak has posted comments on this change. Change subject: [WIP] Remove logging dependencies ......................................................................
Patch Set 4: Code-Review-1 Juan, 1. the logging framework automatically picked-up by appache httpclient when configured, and all it activity logged automatically when debug=true 2. that's true that developer can configure logging by himself, but from my experience most of sdk users are not developers, but IT personnel working on small integrations, in most cases they don't know how to do it, and/or not willing getting to this adventure, so while you can actively debug the application, you won't want doing that or won't be able doing it as you won't have access to customer's site, while they can easily send you logs ..., thus i don't think you should be removing log4j without providing alternatives, after all this feature will serve you (or any other sdk maintainer) when complains arrive. -- To view, visit http://gerrit.ovirt.org/34900 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id81d1490f9f5594e8065b8a20c71d9595d35814b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8...@yahoo.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
