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 <mkub...@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 <mkub...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to