Change in ovirt-engine[master]: core: ActivateDeactivateSingleAsyncOperationFactory- wrong type

Liron Aravot has submitted this change and it was merged. Change subject: core: ActivateDeactivateSingleAsyncOperationFactory- wrong type ...................................................................... core: ActivateDeactivateSingleAsyncOperationFactory- wrong type ActivateDeactivateSingleAsyncOperationFactory expects the recieved VDS list to be an ArrayList. That list is usually passed as returned by the DAO, which means that the concrete type depends on the spring jdbc internal implementation. In this patch I create an ArrayList out of the passed List, I didn't change the type as the entire "Operation" infrastracture use that list for different operation by issuing get() (I assume that this is the reason ArrayList was used in the first place). We need to make further changes here, but this is a quick fixup. As that infrastracture isn't used so commonly, this solution seems good enough in the meanwhile. Change-Id: If0de8b0f5838e7a464cf093b2deeeda67b32a560 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1177595 Signed-off-by: Liron Aravot <laravot(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateDeactivateSingleAsyncOperationFactory.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Liron Aravot: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36525 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0de8b0f5838e7a464cf093b2deeeda67b32a560 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org