Gilad Chaplik has posted comments on this change.

Change subject: restapi: NUMA feature restful API support
......................................................................


Patch Set 9: Code-Review-1

(2 comments)

http://gerrit.ovirt.org/#/c/26943/9/backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmNumaNodesResource.java
File 
backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmNumaNodesResource.java:

Line 28:     @DELETE
Line 29:     @Path("{id}")
Line 30:     public Response remove(@PathParam("id") String id);
Line 31: 
Line 32:     @Path("{iden}")
iden?
Line 33:     public VmNumaNodeResource 
getVmNumaNodeSubResource(@PathParam("iden") String id);


http://gerrit.ovirt.org/#/c/26943/9/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd:

Line 4281:           <jaxb:property generateIsSetMethod="false"/>
Line 4282:         </xs:appinfo>
Line 4283:       </xs:annotation>
Line 4284:     </xs:attribute>
Line 4285:     <xs:attribute name="index" type="xs:int" use="required"/>
no need a ref to HostNuma?
Line 4286:   </xs:complexType>
Line 4287: 
Line 4288:   <xs:complexType name="Cores">
Line 4289:     <xs:sequence>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I72ed4b16c220decbf640f74c4aadffe423afc290
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jason Liao <chuan.l...@hp.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Jason Liao <chuan.l...@hp.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Xiaolei Shi <xiao-lei....@hp.com>
Gerrit-Reviewer: automat...@ovirt.org
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

Reply via email to