Problem/Motivation

#2902607: Hard limit does not work for facets added support for the "Hard limit" facet option that allows you to limit the number of facet values that are returned in the search result. However, it lacks support for the "Hard limit" value of "No limit", or 0. In this case, the number of facet values should be unlimited, however in reality it get limited to 10, the Elasticsearch default.

Proposed resolution

In the case when "No limit" is selected, use a large integer as the size sent to Elasticsearch to effectively get the same result as if Elasticsearch actually supported no limit on the number of facets returned.

Remaining tasks

Patch and review.

User interface changes

None.

API changes

Size sent to Elasticsearch for facets configured for no limit will be a very large integer.

Data model changes

None.

Comments

m4olivei created an issue. See original summary.

m4olivei’s picture

Assigned: m4olivei » Unassigned
Status: Active » Needs review

  • skek authored 6157a9c on 8.x-5.x
    Merge pull request #22 from m4olivei/2911858-facets-no-limit
    
    Issue #...
  • m4olivei committed bf999b1 on 8.x-5.x
    Issue #2911858 by m4olivei: Fix missing support for a Hard limit value...

skek credited skek.

skek’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.