Juan Hernandez has posted comments on this change.

Change subject: restapi: Add a default value for SPM priority (#808629)
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
Line 1063:         </xs:attribute>
Unfortunately there are two problems with the solution you propose:

1. Apparently the JAXB compiler is ignoring this annotation, maybe it is 
because the StorageManager type is defined in a not that usual way, quite 
different to other types in the API.

2. Parts of the code are using the isSet method, so that would require changes 
as well (if the JAXB compiler worked correctly).

Any suggestion?

--
To view, visit http://gerrit.ovirt.org/3476
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I50f22ebd291601a95af084f75c5ab182b3b67b36
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to