Tomáš Došek has posted comments on this change.

Change subject: engine: Fix for RHBZ#968958 - Refactor unnecessary while loop
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
Line 117:                         }
Line 118:                     } else {
Line 119:                         // Print user GUID and another logging info 
only if it was not printed previously already
Line 120:                         logQueryContext(userGuid.toString(), 
uri.toString(), currentLdapServer);
Line 121:                     }
Disregard my previous comment me-- should have looked twice ;)
Line 122:                     System.out.println("No user in Directory was 
found for " + userName
Line 123:                             + ". Trying next LDAP server in list");
Line 124:                 } else {
Line 125:                     System.out.println(InstallerConstants.ERROR_PREFIX


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I938726d631ac4ccb91cc68791e8e349dd43895ea
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomáš Došek <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Tomáš Došek <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to