Sergey Gotliv has uploaded a new change for review.

Change subject: engine: Calculate 'procedureName' just once.
......................................................................

engine: Calculate 'procedureName' just once.

Change-Id: Ia79e6b239f99416e4d2e21cf44d002f1447e9b70
Signed-off-by: Sergey Gotliv <sgot...@redhat.com>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/31/19231/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
index 12bb977..7ac6f0e 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
@@ -35,8 +35,11 @@
      * In case this parameter is null the default procedure is used.
      */
     public void updateAll(String procedureName, Collection<T> entities) {
+        if (procedureName == null) {
+            procedureName = getProcedureNameForUpdate();
+        }
         for (T entity : entities) {
-            update(entity, procedureName == null ? getProcedureNameForUpdate() 
: procedureName);
+            update(entity, procedureName);
         }
     }
 


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

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

Reply via email to