Michael Pasternak has posted comments on this change. Change subject: restapi: default version to 'general' ......................................................................
Patch Set 3: Hi Einav, if you want to force users specifying version, leave only one constructor and replace 'String version' with 'Version version', in addition maybe we should add validation to make sure version get specified, current solution is only temporary resolution of the problem as you have no tool to prevent someone passing NULL as version tomorrow. -- To view, visit http://gerrit.ovirt.org/11173 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I946c9925f3e0b94a9dad86f0958c2f9e1d0ab5c8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <[email protected]> Gerrit-Reviewer: Alexander Wels <[email protected]> Gerrit-Reviewer: Einav Cohen <[email protected]> Gerrit-Reviewer: Michael Pasternak <[email protected]> Gerrit-Reviewer: Ori Liel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
