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

Help search: Make the number of max hits before filtering configurable #1737

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

howlger
Copy link
Contributor

@howlger howlger commented Feb 11, 2025

In the search of the help system, make the number of topics to be searched for before the filtering is applied configurable, so that it can be set in the plugin_customization.ini file.

If the number is too small, hits may be missed, e.g. when searching in a topic and its subtopics. The higher the number, the slightly worse the performance. The default is 1,000 and the maximum number of hits displayed to the user is 500, which means that if more than 500 of the hits before filtering are outside of the search scope, some hits will be missed.

See also: Eclipse Bug 357857 - [Help][Search] "Search selected topic and subtopics" sometimes shows "Nothing found" by mistake

In the search of the help system, make the number of topics to be
searched for before the filtering is applied configurable, so that it
can be set in the "plugin_customization.ini" file.

If the number is too small, hits may be missed, e.g. when searching in a
topic and its subtopics. The higher the number, the slightly worse the
performance. The default is 1,000 and the maximum number of hits
displayed to the user is 500, which means that if more than 500 of the
hits before filtering are outside of the search scope, some hits will be
missed.

See also:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=357857
Copy link
Contributor

Test Results

 1 758 files  ±0   1 758 suites  ±0   1h 28m 21s ⏱️ + 3m 5s
 4 174 tests ±0   4 151 ✅ ±0   23 💤 ±0  0 ❌ ±0 
13 119 runs  ±0  12 952 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit 069a79f. ± Comparison against base commit 0fc54b8.

@iloveeclipse iloveeclipse merged commit dc18b24 into eclipse-platform:master Feb 11, 2025
17 checks passed
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.

2 participants