Laszlo Hornyak has submitted this change and it was merged.

Change subject: core: Remove GetAllEventSubsribers
......................................................................


core: Remove GetAllEventSubsribers

GetAllEventSubsribersQuery is not used anywhere in
the code, and hence should be removed.

This patch removes the following:
1. The GetAllEventSubsribersQuery class.
2. The GetAllEventSubsribers constant representing
   it in VdcQueryType.
3. The getAll() method in EvetDAO, its
   implementation, test and underlying stored procedure, as this
   method was only used by the aforementioned query.

Change-Id: Ic49e45cfdec1b08d2688d1e33f62a32176510547
Signed-off-by: Allon Mureinik <[email protected]>
---
M backend/manager/dbscripts/event_sp.sql
D 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllEventSubscribersQuery.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAO.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAODbFacadeImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/EventDAOTest.java
6 files changed, 0 insertions(+), 59 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic49e45cfdec1b08d2688d1e33f62a32176510547
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to