Change in ovirt-engine[master]: engine: Removing all unneeded staff from VdsIdVDSCommandBase

mkublin at redhat.com mkublin at redhat.com
Sun Oct 14 08:28:58 UTC 2012


Michael Kublin has submitted this change and it was merged.

Change subject: engine: Removing all unneeded staff from VdsIdVDSCommandBase
......................................................................


engine: Removing all unneeded staff from VdsIdVDSCommandBase

1. Removed garbage comments
2. Removed method which is creates quartz task for update vds dynamic data
3. Removed call for that method from CreateVmVDSCommand - call is unneeded
   similar update is done at next line
The patch is not only cleans, it is also reduced unneeded update query and
quartz task creation on every run vm

Change-Id: Ida4f92f70d24634e3fb76d675ba81c936a487288
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsIdVDSCommandBase.java
2 files changed, 0 insertions(+), 58 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ida4f92f70d24634e3fb76d675ba81c936a487288
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list