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