Liron Aravot has uploaded a new change for review.

Change subject: core: ovf  db updates depends on config values
......................................................................

core: ovf  db updates depends on config values

When the config value OvfItemsCountPerUpdate is higher than the constant
OVF_MAX_ITEMS_PER_SQL_STATEMENT which defaults to 100 and there are more
then 100 OVFs to update, not all the data will be updated to the db
because the lists are cleared after the first sql update operation.

Change-Id: If978c8369178ad460419f1ac8ab630184e98287c
Signed-off-by: lara...@redhat.com <lara...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/09/38509/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java
index 2c611ec..8cb8d7f 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java
@@ -209,8 +209,8 @@
             List<String> ovfConfigurationsInfo = 
proccessedOvfConfigurationsInfo.subList(i, i + sizeToUpdate);
             
getVmAndTemplatesGenerationsDAO().updateOvfGenerations(guidsForUpdate, 
ovfGenerationsForUpdate, ovfConfigurationsInfo);
             i += sizeToUpdate;
-            initProcessedInfoLists();
         }
+        initProcessedInfoLists();
     }
 
     /**


-- 
To view, visit https://gerrit.ovirt.org/38509
To unsubscribe, visit https://gerrit.ovirt.org/settings

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

Reply via email to