Liron Aravot has submitted this change and it was merged. Change subject: core: VdsDao - adding getAllForStoragePoolAndStatuses() ...................................................................... core: VdsDao - adding getAllForStoragePoolAndStatuses() This patch adds getAllForStoragePoolAndStatuses() to VdsDAO to provide abillity to get hosts in different statuses in one db call instead of calling getAllForStoragePoolAndStatus() multiple times and/or perform filtering on the calling side. Change-Id: I5984d5e683b8ae18268248cedb22c1ea66987037 Signed-off-by: Liron Aravot <laravot@redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/IdentifiableUtils.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDaoTest.java M packaging/dbscripts/create_functions.sql M packaging/dbscripts/vds_sp.sql 6 files changed, 69 insertions(+), 11 deletions(-) Approvals: Liron Aravot: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/45357 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5984d5e683b8ae18268248cedb22c1ea66987037 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Ala Hino <ahino@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>