710 is missing from the default mappings for author-name-corporate (it currently contains 110, 111, 711, 810, and 811). This is clearly a simple omission.
Created attachment 121127 [details] [review] Bug 28379: Elasticsearch - Add 710 to author-name-corporate index (MARC21) This patch adds 710 to the default author-name-corporate index mappings for elasticsearch. To test: 1) Apply patch 2) Reindex using rebuild_elasticsearch.pl -r If you don't have access to a terminal (in a sandbox for example) 2a) Go to Administration > Search engine configuration, click "Reset mappings" and confirm 2b) Then reindex 3) Search for an author name found only in 710 using the Author (Corporate name) index in advanced search
Created attachment 123589 [details] [review] Bug 28379: Elasticsearch - Add 710 to author-name-corporate index (MARC21) Signed-off-by: David Nind <david@davidnind.com>
Hi Caroline, be careful - you cannot add a commit message with git bz attach, you need to do it with git commit or git commit --amend, otherwise it won't become part of the patch file, but only displays on bugzilla. Fixed here and on another patch.
Created attachment 123678 [details] [review] Bug 28379: Elasticsearch - Add 710 to author-name-corporate index (MARC21) This patch adds 710 to the default author-name-corporate index mappings for elasticsearch. To test: 1) Apply patch 2) Reindex using rebuild_elasticsearch.pl -r If you don't have access to a terminal (in a sandbox for example) 2a) Go to Administration > Search engine configuration, click "Reset mappings" and confirm 2b) Then reindex 3) Search for an author name found only in 710 using the Author (Corporate name) index in advanced search Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 21.11, thanks to everybody involved!