Change in ovirt-engine[master]: core: improve active iso domain lookup

[ofrenkel at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server