Change in ovirt-engine[master]: core: Inline RoleGroupMapsRowMapper
amureini at redhat.com
amureini at redhat.com
Thu Jul 4 08:48:54 UTC 2013
Allon Mureinik has submitted this change and it was merged.
Change subject: core: Inline RoleGroupMapsRowMapper
......................................................................
core: Inline RoleGroupMapsRowMapper
Inlined RoleGroupMapsRowMapper in RoleGroupMapsDAODbFacadeImpl as per
project conventions.
Additionally, this refactoring will allow the RoleGroupMapsRowMapper
to benefit from the utilities present in BaseDbFacadeDAO, such as
Guid mapping (which will be implemented in a subsequent patch).
Change-Id: I541535b86dc80279ee95e63751e4395000ea11d1
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
D 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/RoleGroupMapDAODbFacadeImpl.java
2 files changed, 13 insertions(+), 21 deletions(-)
Approvals:
Allon Mureinik: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/16391
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I541535b86dc80279ee95e63751e4395000ea11d1
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list