Moti Asayag has uploaded a new change for review.

Change subject: engine: Keep code DRY
......................................................................

engine: Keep code DRY

Reuse same dao implementation to prevent code duplications.

Change-Id: I52f52c292a0c1eedff84108d325a1adf17676525
Signed-off-by: Moti Asayag <masa...@redhat.com>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
1 file changed, 1 insertion(+), 8 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/41/36241/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
index 6cf1645..5eb57fa 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
@@ -63,14 +63,7 @@
 
     @Override
     public VDSGroup getByName(String name) {
-        MapSqlParameterSource parameterSource = 
getCustomMapSqlParameterSource()
-                .addValue("vds_group_name", name)
-                .addValue("is_case_sensitive", true);
-
-        return (VDSGroup) DbFacadeUtils.asSingleResult(
-                getCallsHandler().executeReadList("GetVdsGroupByVdsGroupName",
-                        VdsGroupRowMapper.instance,
-                        parameterSource));
+        return (VDSGroup) DbFacadeUtils.asSingleResult(getByName(name, true));
     }
 
     @Override


-- 
To view, visit http://gerrit.ovirt.org/36241
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I52f52c292a0c1eedff84108d325a1adf17676525
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to