Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add term size config #852

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Add term size config #852

wants to merge 3 commits into from

Conversation

haideriqbal
Copy link
Collaborator

@haideriqbal haideriqbal commented Feb 14, 2025

This fixes: #810

Here hows the functionality has changed.

Anyone running their own instance of OLS can provide the param in application.properties. The value of param will be treated as upper limit on the number of rows fetched from solr. For example if the param value is 5000 then

  • If no size param is defined in the URL then by default 5000 rows would be fetched in the API call
  • If size param is specified in the URL e.g. 100 and the size is less than the value of config param then the 100 rows would be fetched
  • If size param is specified in the URL e.g. 20000 and the size is greater than the value of config param then 5000 rows would be fetched.

In case of OLS, config param is optional and its default value is 1000.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Config option to change hard limit on term retrieval
1 participant