[Engine-patches] Change in ovirt-engine[master]: backend, webadmin: storage tab is sluggish (#769615)

gchaplik at redhat.com gchaplik at redhat.com
Sun Jan 29 04:29:17 EST 2012


Gilad Chaplik has submitted this change and it was merged.

Change subject: backend, webadmin: storage tab is sluggish (#769615)
......................................................................


backend, webadmin: storage tab is sluggish (#769615)

https://bugzilla.redhat.com/769615

it seems that the concatenation and string format of
storage BE vdcQueryableId consumes alot of cpu (in FF).
now it returns only the id, instead of constructing new id (that contains DC id),
for iso purposes.
I moved the logic to the ui (Storage - DataCenter sub tab)

Change-Id: I49d9fada489168f093deb273e8b9510146f58522
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage_domains.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java
2 files changed, 25 insertions(+), 4 deletions(-)

Approvals:
  Michael Kublin: Looks good to me, approved
  Gilad Chaplik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I49d9fada489168f093deb273e8b9510146f58522
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Danny Rankevich <danny.ran at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>


More information about the Engine-patches mailing list