Omer Frenkel has submitted this change and it was merged.

Change subject: core: add distinct keyword to search
......................................................................


core: add distinct keyword to search

currently, the search can return duplicate results,
for example when template has more than one disk,
or disks are on different storage domains.

this patch adds a 'distinct' to the sql the search mechanism creates,
in order to remove any duplicates that return from the query.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1130915
Change-Id: I070900dc9e1064c98f8b342f26dbc3c23ba8c4a4
Signed-off-by: Omer Frenkel <ofren...@redhat.com>
---
M 
backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java
M 
backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java
2 files changed, 67 insertions(+), 65 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I070900dc9e1064c98f8b342f26dbc3c23ba8c4a4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzel...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to