Bugzilla – Attachment 135751 Details for
Bug 30882
Add max_result_window to index config
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 30882: Add max_result_window to index config
Bug-30882-Add-maxresultwindow-to-index-config.patch (text/plain), 1.80 KB, created by
David Nind
on 2022-06-07 10:32:39 UTC
(
hide
)
Description:
Bug 30882: Add max_result_window to index config
Filename:
MIME Type:
Creator:
David Nind
Created:
2022-06-07 10:32:39 UTC
Size:
1.80 KB
patch
obsolete
>From 0676bf56160af1f298ee97a1393a84aa55ceb872 Mon Sep 17 00:00:00 2001 >From: Fridolin Somers <fridolin.somers@biblibre.com> >Date: Wed, 1 Jun 2022 20:24:03 -1000 >Subject: [PATCH] Bug 30882: Add max_result_window to index config > >Elasticsearch number of results is by default limited by setting "index.max-result-window", default value is 10000. >https://www.elastic.co/guide/en/elasticsearch/reference/current/index-modules.html#index-max-result-window > >We use this setting: >https://git.koha-community.org/Koha-community/Koha/src/commit/44d6528b566e5c16eba9b15f8aa24667293766c3/Koha/SearchEngine/Elasticsearch/Search.pm#L411 > >I propose we add this setting in index config. > >Test plan: >1) Use Elasticsearch >2) Apply patch and flush memcached >3) Rebuild indexes: misc/search_tools/rebuild_elasticsearch.pl -v -b -d >4) Check the settings of index (when using koha-testing-docker*): > curl 'es:9200/koha_kohadev_biblios/_settings?pretty&filter_path=**.max_result_window' >5) You should see: > "max_result_window" : "1000000" > >* You also need to add this setting to the es section in koha-testing-docker's >docker-compose.yml (after the networks configuration): > ports: > - "9200:9300" > >Signed-off-by: David Nind <david@davidnind.com> >--- > admin/searchengine/elasticsearch/index_config.yaml | 1 + > 1 file changed, 1 insertion(+) > >diff --git a/admin/searchengine/elasticsearch/index_config.yaml b/admin/searchengine/elasticsearch/index_config.yaml >index b8b2ac6071..b53578fef0 100644 >--- a/admin/searchengine/elasticsearch/index_config.yaml >+++ b/admin/searchengine/elasticsearch/index_config.yaml >@@ -42,5 +42,6 @@ index: > pattern: '\s*(?<!\p{Lu})[.\-,;]*\s*$' > replacement: '' > index.mapping.total_fields.limit: 10000 >+index.max_result_window: 1000000 > index.number_of_replicas: 1 > index.number_of_shards: 5 >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 30882
:
135572
|
135751
|
137445
|
137834