Change in ovirt-engine[master]: core: remove pending VMs counting

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Dec 2 09:40:35 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: remove pending VMs counting
......................................................................


core: remove pending VMs counting

The number of pending VMs is not in use anywhere, but it is still
computed. In order to compute the pending VMs we increment and decrement
the counter while running VMs in synchronized blocks and it is not good
in terms of performance. So since this code is redundant, it is better
to remove it.

Change-Id: I3bd7a757be2979b86227ce56064e6165080035eb
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java
2 files changed, 0 insertions(+), 24 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3bd7a757be2979b86227ce56064e6165080035eb
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list