Allon Mureinik has submitted this change and it was merged. Change subject: core: rv GetEntityPermissionsi[ForUserAndGroups] ......................................................................
core: rv GetEntityPermissionsi[ForUserAndGroups] Change get_entity_permissions and get_entity_permissions_for_user_and_groups to use a return value instead of an output parameter. Although the main reasoning behind this is standardisation and code cleanliness, there is an added performance bonus as this change removes an explicit conversion of the returned uuid to a varchar. Change-Id: I26d94fc765c927763af6bb0fa6775b4aaced6a28 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/PermissionDAODbFacadeImpl.java M packaging/dbscripts/multi_level_administration_sp.sql 2 files changed, 15 insertions(+), 25 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26d94fc765c927763af6bb0fa6775b4aaced6a28 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tal Nisan <[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
