Change in ovirt-engine[master]: core: Extract common NetworkStatistics DAO code

lvernia at redhat.com lvernia at redhat.com
Wed Feb 11 09:16:54 UTC 2015


Lior Vernia has submitted this change and it was merged.

Change subject: core: Extract common NetworkStatistics DAO code
......................................................................


core: Extract common NetworkStatistics DAO code

The same code was duplicated all over different DAOs (for host
interfaces, VM interfaces, VM interface statistics). It's now
extracted and re-used.

Change-Id: Ifc60a5b80d7ef042b1b81200bb8542081b42acc0
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/CustomMapSqlParameterSource.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDaoDbFacadeImpl.java
A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/NetworkStatisticsDaoDbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNetworkInterfaceDaoDbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNetworkStatisticsDaoDbFacadeImpl.java
5 files changed, 114 insertions(+), 54 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifc60a5b80d7ef042b1b81200bb8542081b42acc0
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list