Juan Hernandez has posted comments on this change. Change subject: aaa: Introducing changes to API (domain groups/ids) ......................................................................
Patch Set 9: (1 comment) http://gerrit.ovirt.org/#/c/26191/9/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/utils/DirectoryEntryIdUtils.java File backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/utils/DirectoryEntryIdUtils.java: Line 1: package org.ovirt.engine.api.restapi.utils; Line 2: Line 3: import java.io.UnsupportedEncodingException; Line 4: Line 5: public class DirectoryEntryIdUtils { I see that you these methods are inspired in the hex conversion methods of the now removed ExternalId class. Those methods where implemented this way in order to avoid external dependencies (on commons-codec) in a class that was potentially part of the contracts of the engine. Also the "decode" method was intended to silently ignore anything that isn't hex characters, in order to be able to decode also UUID representations. None of these things are required now, so I think you should better use the hex support in commons-codec. Line 6: Line 7: public static String encode(String source) { Line 8: try { Line 9: byte[] bytes; -- To view, visit http://gerrit.ovirt.org/26191 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iaecb5d43945769db82475edde1c7075c1a343c07 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
