Maor Lipchuk has submitted this change and it was merged. Change subject: core: Filtering VdsGroupDAO.getAllForStoragePool ...................................................................... core: Filtering VdsGroupDAO.getAllForStoragePool Added an (overloaded) filtered verion of VdsGroupDAO.getAllForStoragePool, to be used by queries in the User Portal. This patch includes the postgres implementation, the java DAO that calls it and a test that proves it functions properly. Change-Id: I2e1e1063007b57ec3bdaeed0abc832b477f906dc Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/dbscripts/vds_groups_sp.sql M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAOHibernateImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsGroupDAOTest.java 5 files changed, 73 insertions(+), 6 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, but someone else must approve Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4365 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e1e1063007b57ec3bdaeed0abc832b477f906dc Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>