Omer Frenkel has submitted this change and it was merged. Change subject: core: improve active iso domain lookup ......................................................................
core: improve active iso domain lookup The lookup for active iso domain at IsoDomainListSyncronizer#findActiveISODomain access the DB twice: at first to fetch all the domains in the given storage pool and then it access the DB again to fetch a given domain within the same storate pool. the second access seems to be redundant since the fetched domain already exists in the list that was return from the first query, thus the second query is removed. Change-Id: I8859a85107f8c2e658c23d6a04b69d0201d086d5 Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java 1 file changed, 4 insertions(+), 9 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17578 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8859a85107f8c2e658c23d6a04b69d0201d086d5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
