[Bug 27206] New: Elasticsearch sort by popularity must use totalissues search field
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Bug ID: 27206 Summary: Elasticsearch sort by popularity must use totalissues search field Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Searching - Elasticsearch Assignee: koha-bugs@lists.koha-community.org Reporter: fridolin.somers@biblibre.com With Zebra search engine, sort by popularity uses search field 9003. This is from bib1.att : att 9003 totalissues Actually Elasticsearch sort by popularity uses search field 'issues' : https://git.koha-community.org/Koha-community/Koha/src/commit/cae0ef76f398bb... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |fridolin.somers@biblibre.co |ity.org |m -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 --- Comment #1 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 114344 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114344&action=edit Bug 27206: Elasticsearch sort by popularity must use totalissues search field With Zebra search engine, sort by popularity uses search field 9003. This is from bib1.att : att 9003 totalissues 'totalissues' beeing on 942$0 Actually Elasticsearch sort by popularity uses search field 'issues' : https://git.koha-community.org/Koha-community/Koha/src/commit/cae0ef76f398bb... Test plan : 1) Use zebra search engine 2) Perform some checkouts and checkins 3) Run misc/cronjobs/update_totalissues.pl 4) Perform a search sorting on popularity 5) Change search engine to Elasticsearch 6) Rebuild all biblios : misc/search_tools/rebuild_elasticsearch.pl -b 7) Perform same search sorting on popularity => You should get same order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion --- Comment #2 from Nick Clemens <nick@bywatersolutions.com> --- Elastic uses the sum of the 'issues' field which is set to the default of '952l' - so it should match the output when the totalissues field is counted using items. If you want it to match totalissues using statistics table, then we could map it to 942$0 Using the sum of the issues from items means we don't rely on the cronjob to update_totalissues which is maybe a good thing? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 --- Comment #3 from Fridolin Somers <fridolin.somers@biblibre.com> --- (In reply to Nick Clemens from comment #2)
Elastic uses the sum of the 'issues' field which is set to the default of '952l' - so it should match the output when the totalissues field is counted using items. Ah sure. But I don't see where Elastic does this sum ?
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 --- Comment #4 from Nick Clemens <nick@bywatersolutions.com> --- (In reply to Fridolin Somers from comment #3)
(In reply to Nick Clemens from comment #2)
Elastic uses the sum of the 'issues' field which is set to the default of '952l' - so it should match the output when the totalissues field is counted using items. Ah sure. But I don't see where Elastic does this sum ?
https://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=Koha/SearchEngine... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27206 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |RESOLVED Resolution|--- |INVALID --- Comment #5 from Fridolin Somers <fridolin.somers@biblibre.com> --- Ohhhh thanks a lot. Field issues is indeed type sum. This is for the best ;) I mark this as invalid. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org