Change in ovirt-engine[master]: core: Filtering VdsGroupDAO.getAllForStoragePool

mlipchuk at redhat.com mlipchuk at redhat.com
Tue May 15 14:42:21 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list