Maor Lipchuk has posted comments on this change. Change subject: core: Add storage types to DiskImage entity ......................................................................
Patch Set 8: (1 comment) http://gerrit.ovirt.org/#/c/23410/8/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskImageDAODbFacadeImpl.java File backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskImageDAODbFacadeImpl.java: Line 192: for (String typeStr : splitTypes) { Line 193: try { Line 194: types.add(StorageType.forValue(Integer.parseInt(typeStr))); Line 195: } Line 196: catch (NumberFormatException e) { > NumberFormatException can be thrown in case there is an error in the parsin DB data should be valid, so it should not happened, and there is no point to put the catch. I don't think this issue it is that relevant, though I would recommend to remove this. Line 197: throw new SQLException("Could not parse disk image storage domain type " + typeStr, e); Line 198: } Line 199: } Line 200: return types; -- To view, visit http://gerrit.ovirt.org/23410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8302972bb21fa076cd35fb308f5f660ca9200daa Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
