Moti Asayag has submitted this change and it was merged. Change subject: engine: Refactor InterfaceDaoDbFacadeImpl ...................................................................... engine: Refactor InterfaceDaoDbFacadeImpl Reduced amount of duplicated code, cleared eclipse warnings and mostly shortening this class. There is still way to go: changing this class to inherit from DefaultGenericDaoDbFacade. Change-Id: I4a46cf2ea27c163ab5ed4fd7c03be37314d9e354 Signed-off-by: Moti Asayag <masayag(a)redhat.com&gt; --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDaoDbFacadeImpl.java 1 file changed, 38 insertions(+), 88 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/22636 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a46cf2ea27c163ab5ed4fd7c03be37314d9e354 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com&gt; Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: oVirt Jenkins CI Server