Change in ovirt-engine[master]: core: improve active iso domain lookup
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 <ahadas(a)redhat.com>
---
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 <ahadas(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server