Ori Liel has posted comments on this change.

Change subject: restapi: Enable search on 2nd-level collections - WIP
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/38371/2/backend/manager/modules/restapi/interface/definition/pom.xml
File backend/manager/modules/restapi/interface/definition/pom.xml:

Line 17:      <dependency>
Line 18:        <groupId>org.ovirt.engine.core</groupId>
Line 19:        <artifactId>searchbackend</artifactId>
Line 20:        <version>${engine.version}</version>
Line 21:      </dependency>
> I think this isn't needed, as you aren't doing any other change to the rest
you're right, the new Enum I added was first put in definition project and 
later I realized it's a better fit for interface-common-jaxrs, and I didn't 
change the pom dependency to suit that. Will do.
Line 22: 
Line 23:     <dependency>
Line 24:       <groupId>org.jboss.resteasy</groupId>
Line 25:       <artifactId>resteasy-jaxrs</artifactId>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3edd6ce6dbef351a0ba00b1a7a4c67476e67f778
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.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