Oved Ourfali has submitted this change and it was merged. Change subject: core: Replaced the check of whether a storage pool contains mixed type ......................................................................
core: Replaced the check of whether a storage pool contains mixed type Replaced the check of whether a storage pool contains storage domains of mixed types to use a designated query that fetches distinct storage types instead of fetching all the storage domains and filtering by that, in that way we are fetching less data and eliminating the use for logic in engine to determine if the domain is mixed thus resulting in better performance Change-Id: I86cf1512655865831de3cbd16c10a7bc243f08cd Bug-Url: https://bugzilla.redhat.com/1083560 Signed-off-by: Tal Nisan <tni...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBaseTest.java 2 files changed, 8 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27138 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86cf1512655865831de3cbd16c10a7bc243f08cd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches