Change in ovirt-engine[master]: core: Modify constructor to be compatible with caller

yzaslavs at redhat.com yzaslavs at redhat.com
Fri Jun 27 02:22:03 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Modify constructor to be compatible with caller
......................................................................


core: Modify constructor to be compatible with caller

ProcessOvfUpdateForStorageDomainCommand is being called without context,
although the constructor is only supported with context parameter, so we
will always get runtime exception when being called.

The proposed fix remove the context from the
ProcessOvfUpdateForStorageDomainCommand constructor

Change-Id: Id4c1c7f36876b51d2194e69b95a3d7c20f43b659
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Maor Lipchuk: Verified
  Yair Zaslavsky: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id4c1c7f36876b51d2194e69b95a3d7c20f43b659
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list