Change in ovirt-engine[ovirt-engine-3.5]: core: add distinct keyword to search

tnisan at redhat.com tnisan at redhat.com
Wed Sep 17 15:07:15 UTC 2014


Tal Nisan 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 <ofrenkel at 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:
  Eli Mesika: Looks good to me, approved
  Omer Frenkel: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I070900dc9e1064c98f8b342f26dbc3c23ba8c4a4
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list