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(a)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(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>