Allon Mureinik has uploaded a new change for review. Change subject: core: Cleanup RoleGroupMapDAO and its tests ......................................................................
core: Cleanup RoleGroupMapDAO and its tests 1. Used a static instance of the row mapper as per http://ovirt.org/wiki/Backend_Coding_Standards. 2. Replaced executeReadAndReturnMap calls to the standard executeRead and executeReadList. 2. Replaed * imports with FCQN imports. Change-Id: I63a99b42978955b1c8b82ec0849f095de07c1158 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/RoleGroupMapRowMapper.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/RoleGroupMapDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/RoleGroupMapDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/RoleGroupMapDAOTest.java 4 files changed, 17 insertions(+), 29 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/72/4472/1 -- To view, visit http://gerrit.ovirt.org/4472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I63a99b42978955b1c8b82ec0849f095de07c1158 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
