Martin Peřina has posted comments on this change. Change subject: core: Fixing NPE in case a directory was not found for user. ......................................................................
Patch Set 5: (1 comment) http://gerrit.ovirt.org/#/c/25921/5/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java: Line 87: log.warn(String.format("No profile was found for user %1$s. It is possible that the relevant domain for the user was removed for the user. Marking the user as inactive", Could you please split string message into more line so the line won't exceed 120 chars limit? -- To view, visit http://gerrit.ovirt.org/25921 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ief4761b9234f3839a1dba1eefbf871ae2bae3c69 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches