Maor Lipchuk has posted comments on this change. Change subject: core: Introducing VersionStorageFormatUtil ......................................................................
Patch Set 2: 1. I would consider to add a StorageFormatType as a private parameter in the version class. We can benefit the following: 1. We will avoid adding a new map class which we will need to maintain in the future. 2. Developers will not forget to add the storage format type on each new version. 3. We can use default value for StorageFormatType. 2. Please also change AddStoragePoolWithStoragesCommand accordingly. -- To view, visit http://gerrit.ovirt.org/18697 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I47fb8b14e5c97b601c8fc0cc296a4ec42a48227a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
