Allon Mureinik has uploaded a new change for review.

Change subject: core: Use VdsRowMapper static instance
......................................................................

core: Use VdsRowMapper static instance

Use VdsRowMapper.instance wherever possible instead of creating a new
instance.

Change-Id: I7baed640aafe8c9ebd25c722609192cd88196f93
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/75/12575/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
index c585764..b538e4c 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
@@ -149,7 +149,7 @@
     @Override
     public List<VDS> getAllForVdsGroupWithStatus(Guid vdsGroupId, VDSStatus 
status) {
         return getCallsHandler().executeReadList("getVdsForVdsGroupWithStatus",
-                new VdsRowMapper(),
+                VdsRowMapper.instance,
                 getCustomMapSqlParameterSource()
                         .addValue("vds_group_id", vdsGroupId)
                         .addValue("status", status.getValue()));
@@ -158,7 +158,7 @@
     @Override
     public List<VDS> getAllForStoragePoolAndStatus(Guid storagePool, VDSStatus 
status) {
         return 
getCallsHandler().executeReadList("getVdsByStoragePoolIdWithStatus",
-                new VdsRowMapper(),
+                VdsRowMapper.instance,
                 getCustomMapSqlParameterSource()
                         .addValue("storage_pool_id", storagePool)
                         .addValue("status", status.getValue()));


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7baed640aafe8c9ebd25c722609192cd88196f93
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

Reply via email to