Change in ovirt-engine[master]: core: fix DAO test falure

masayag at redhat.com masayag at redhat.com
Thu Jun 9 04:31:37 UTC 2016


Moti Asayag has submitted this change and it was merged.

Change subject: core: fix DAO test falure
......................................................................


core: fix DAO test falure

The change populates the default value of LEGACY to
Cluster.requiredSwitchTypeForCluster.  That fixes the NPE that is
thrown in MapSqlParameterSource of ClusterDaoImpl that assumes the
field is non-nullable.

Change-Id: I6bf23da75829f79f79e5a160b3f637bc0b04dda4
Signed-off-by: Yevgeny Zaspitsky <yzaspits at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Cluster.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved
  Yevgeny Zaspitsky: Verified



-- 
To view, visit https://gerrit.ovirt.org/58876
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6bf23da75829f79f79e5a160b3f637bc0b04dda4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list