Sergey Gotliv has posted comments on this change. Change subject: engine, dao: Introduce ISCSI Bond entity, tables and dao ......................................................................
Patch Set 4: (3 comments) .................................................... File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IscsiBond.java Line 103: int result = 1; Line 104: result = prime * result + ((id == null) ? 0 : id.hashCode()); Line 105: result = prime * result + ((name == null) ? 0 : name.hashCode()); Line 106: result = prime * result + ((description == null) ? 0 : description.hashCode()); Line 107: return result; Done Line 108: } Line 109: Line 110: @Override Line 111: public boolean equals(Object obj) { Line 119: Line 120: IscsiBond iscsiBond = (IscsiBond) obj; Line 121: return ObjectUtils.objectsEqual(id, iscsiBond.getId()) && Line 122: ObjectUtils.objectsEqual(name, iscsiBond.getName()) && Line 123: ObjectUtils.objectsEqual(description, iscsiBond.getDescription()); Done Line 124: } .................................................... File backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/IscsiBondDaoDbFacadeImpl.java Line 25: } Line 26: Line 27: @Override Line 28: public List<Guid> getNetworkIdsByIscsiBondId(Guid iscsiBondId) { Line 29: return getCallsHandler().executeReadList("GetNetworksByIscsiBondId", new RowMapper<Guid>() { IscsiBondRowMapper mapping a ResultSet into IscsiBond object. I am not fetching an IscsiBond here, I am fetching a list of guids. Line 30: @Override Line 31: public Guid mapRow(ResultSet rs, int index) throws SQLException { Line 32: return new Guid((UUID) rs.getObject(1)); Line 33: } -- To view, visit http://gerrit.ovirt.org/22951 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I12313c02810a2f0e75016bdd78b44da43f2154d4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches