Ori Liel has posted comments on this change.

Change subject: core: fix e8993e1ca33b823852e15b4adeb3df293afb5676
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/38757/3/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java
File 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java:

Line 152: )
> i don't think Guid is correct. The definition of BusiessEntity doesn't enfo
If you just remove the casting to Guid it should be fine, Ids will be compared 
one to the other using their equals() method.


-- 
To view, visit https://gerrit.ovirt.org/38757
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I23ef5740ea59069b10afc329e5ef75bce3c33b31
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: [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

Reply via email to