From Liron Aravot <laravot@redhat.com>:

Liron Aravot has submitted this change and it was merged. Change subject: core: ProcessOvfUpdateForStoragePoolCommand - db updates ...................................................................... core: ProcessOvfUpdateForStoragePoolCommand - db updates ProcessOvfUpdateForStoragePoolCommand.performOvfUpdate() iterates over the ovf data and updates it to the db, the max limit is for items per update is OVF_MAX_ITEMS_PER_SQL_STATEMENT. The lists are being initiated after the first loop, in case more than one iteration not all the updates will be performed. This case was never actually reached because the number of ids processed is always lower or equals to the OVF_MAX_ITEMS_PER_SQL_STATEMENT - This patch fixes that wrong placement of the initiation for correctness. Change-Id: I968c9cd789a52e66a0c91d33629808e368bc29d1 Signed-off-by: Liron Aravot <laravot@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ovfstore/ProcessOvfUpdateForStoragePoolCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/77313 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I968c9cd789a52e66a0c91d33629808e368bc29d1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>