Description
Nick Clemens (kidclamp)
2019-06-17 17:26:50 UTC
Created attachment 101051 [details] [review] Bug 23137: Add reset option to rebuild_elasticsearch.pl Setup: 1 - Be using Elasticsearch 2 - Reload mappings from the db Admin->Search configuration Reset Mappings 3 - Reindex ES and confirm searching is working To test: 1 - Apply patch 2 - Alter your mappings file for elastic (just change a description for a field) 3 - perl misc/search_tools/rebuild_elasticsearch.pl -r -v Verbose not necessary, but good for letting you know things are progressing 4 - Confirm the mapping change shows in the interface 5 - Confirm reindex worked and searching is working Created attachment 101052 [details] [review] Bug 23137: Add reset and delete to koha-elasticsearch Same as previous test, but use the script to run the reindex reset_all to update the scripts or use misc4dev/cp_debian_files.pl sudo koha-elasticsearch --rebuild -r -v Created attachment 102354 [details] [review] Bug 23137: Add reset option to rebuild_elasticsearch.pl Setup: 1 - Be using Elasticsearch 2 - Reload mappings from the db Admin->Search configuration Reset Mappings 3 - Reindex ES and confirm searching is working To test: 1 - Apply patch 2 - Alter your mappings file for elastic (just change a description for a field) 3 - perl misc/search_tools/rebuild_elasticsearch.pl -r -v Verbose not necessary, but good for letting you know things are progressing 4 - Confirm the mapping change shows in the interface 5 - Confirm reindex worked and searching is working Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Created attachment 102355 [details] [review] Bug 23137: Add reset and delete to koha-elasticsearch Same as previous test, but use the script to run the reindex reset_all to update the scripts or use misc4dev/cp_debian_files.pl sudo koha-elasticsearch --rebuild -r -v Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Please add doc to debian/docs/koha-elasticsearch.xml Created attachment 102918 [details] [review] Bug 23137: Add documentation Should not we have the cache clearing in reset_elasticsearch_mappings? Created attachment 103014 [details] [review] Bug 23137: Move cache flushing to the method Created attachment 103640 [details] [review] Bug 23137: Move cache flushing to the method Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Created attachment 103647 [details] [review] Bug 23137: Add reset option to rebuild_elasticsearch.pl Setup: 1 - Be using Elasticsearch 2 - Reload mappings from the db Admin->Search configuration Reset Mappings 3 - Reindex ES and confirm searching is working To test: 1 - Apply patch 2 - Alter your mappings file for elastic (just change a description for a field) 3 - perl misc/search_tools/rebuild_elasticsearch.pl -r -v Verbose not necessary, but good for letting you know things are progressing 4 - Confirm the mapping change shows in the interface 5 - Confirm reindex worked and searching is working Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 103648 [details] [review] Bug 23137: Add reset and delete to koha-elasticsearch Same as previous test, but use the script to run the reindex reset_all to update the scripts or use misc4dev/cp_debian_files.pl sudo koha-elasticsearch --rebuild -r -v Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 103649 [details] [review] Bug 23137: Add documentation Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 103650 [details] [review] Bug 23137: Move cache flushing to the method Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Nice work everyone! Pushed to master for 20.05 backported to 19.11.x for 19.11.06 Patchset does not apply clean to 19.05.x, no backport |