Change in ovirt-engine[master]: core: VmDevicesMonitoring.Change create ArrayList on demand

ahadas at redhat.com ahadas at redhat.com
Thu Feb 25 12:30:57 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: VmDevicesMonitoring.Change create ArrayList on demand
......................................................................


core: VmDevicesMonitoring.Change create ArrayList on demand

Set lists in VmDevicesMonitoring.Change to null initially and create
them only when some objects are added. Since only part of the lists
will be used at the same time, this change will save memory.

Change-Id: I19ac122578f1d82a5cbabd65d3cb5dd98869524c
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmDevicesMonitoring.java
1 file changed, 76 insertions(+), 23 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Verified
  Arik Hadas: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/53172
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I19ac122578f1d82a5cbabd65d3cb5dd98869524c
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list