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

Reply via email to