[LUCENE-6464] Allow possibility to group contexts in AnalyzingInfixSuggester.loockup()

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.1
• Fix Version/s: 5.2, 6.0
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Arcadius Ahouansou
• Created: 06/May/15 02:06
• Updated: 28/Aug/22 14:32
• Resolved: 15/May/15 15:06

Description

This is an enhancement to LUCENE-6050
LUCENE-6050 added

lookup(CharSequence key, Map<BytesRef, BooleanClause.Occur> contextInfo, int num, boolean allTermsRequired, boolean doHighlight)

which allowed to do something like

(A OR B AND C OR D ...)

In our use-case, we realise that we need grouping i.e
(A OR B) AND (C OR D) AND (...)

In other words, we need the intersection of multiple contexts.

The attached patch allows to pass in a varargs of map, each one representing the each group. Looks a bit heavy IMHO.

This is an initial patch.

The question to mikemccand and janechang is:
is it better to expose a FilteredQuery/Query and let the user build their own query instead of passing a map?

Exposing a filteredQuery will probably give the best flexibility to the end-users.

Issue Links

• is related to: LUCENE-6050 Add possibility to specify SHOUD or MUST for each context for AnalyzingInfixSuggester.loockup()