Change in ovirt-engine[master]: core: don't load disks for templates on search

ofrenkel at redhat.com ofrenkel at redhat.com
Tue Feb 25 13:47:52 UTC 2014


Omer Frenkel has submitted this change and it was merged.

Change subject: core: don't load disks for templates on search
......................................................................


core: don't load disks for templates on search

When searching for templates,
the search loads all the disks for each template.
this is not necessary as disks are a different collection that need to
be fetched upon request,
also the disks are not used currently by the search initiators,
and it make the templates search slow for systems with many templates.

this patch removes the disks population on templates search.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1064907
Change-Id: Ia83a3117f2c65bbb17972095cf3271eb39449cd6
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java
1 file changed, 2 insertions(+), 10 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Omer Frenkel: Verified
  Liran Zelkha: Looks good to me, but someone else must approve
  Arik Hadas: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/24996
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia83a3117f2c65bbb17972095cf3271eb39449cd6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list