Change in ovirt-engine[master]: core: cleanup in MigrateVmCommand

ofrenkel at redhat.com ofrenkel at redhat.com
Tue Aug 27 13:49:06 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: cleanup in MigrateVmCommand
......................................................................


core: cleanup in MigrateVmCommand

Extract the code in MigrateVmCommand#perform method which is related
only to the parameters initialization to separate methods, to make it
shorter and easier to read.

This patch also includes other minor refactoring in this class.

Change-Id: I138ac8c64e7c0fe70c579e80313b56d4d1d47848
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
1 file changed, 42 insertions(+), 52 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I138ac8c64e7c0fe70c579e80313b56d4d1d47848
Gerrit-PatchSet: 4
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: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list