Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup VmDAODbFacadeImpl ......................................................................
core: Cleanup VmDAODbFacadeImpl Use single static instance of the row mapper instead of creating a new for each call, as per the guidelines at http://ovirt.org/wiki/Backend_Coding_Standards. Change-Id: Ifaa0c197c3b376aad96be110640f15b3fe6a7d17 --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java 1 file changed, 28 insertions(+), 24 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Allon Mureinik: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2932 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifaa0c197c3b376aad96be110640f15b3fe6a7d17 Gerrit-PatchSet: 7 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: Livnat Peer <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
