Mike Kolesnik has submitted this change and it was merged.

Change subject: core: DAO to get a VM by it's ID, filtered
......................................................................


core: DAO to get a VM by it's ID, filtered

Added a permission filtered overload for VmDAO.get() and
VmDAO.getById().

Filtering was implemented in the stored procedure level, and
delegated via java.

Change-Id: I4f362b7c17dd580d40546f3cf44018b7e3c0e4b4
---
M backend/manager/dbscripts/vms_sp.sql
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAO.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java
4 files changed, 84 insertions(+), 4 deletions(-)

Approvals:
  Mike Kolesnik: Looks good to me, approved
  Allon Mureinik: Verified


--
To view, visit http://gerrit.ovirt.org/3529
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4f362b7c17dd580d40546f3cf44018b7e3c0e4b4
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to