Make author name matching case insensitive #9390
Labels
Lead: @scottbarnes
Issues overseen by Scott (Community Imports)
Module: Import
Issues related to the configuration or use of importbot and other bulk import systems. [managed]
Needs: Investigation
This issue/PR needs a root-cause analysis to determine a solution. [managed]
Priority: 3
Issues that we can consider at our leisure. [managed]
Type: Feature Request
Issue describes a feature or enhancement we'd like to implement. [managed]
Related: #9003, internetarchive/infogami#221
Problem
A clear and concise description of what you want to happen
On import, author name matching should be case insensitive.
Additional Context
internetarchive/infogami#217 changed
~
to useILIKE
rather thanLIKE
, and the Open Library code in #9003 relied upon this to perform case insensitive author name matching on import.However, the Infogami
ILIKE
change caused performance issues and is slated to be reverted in internetarchive/infogami#221, with~
doing aLIKE
operation and~i
doing anILIKE
operation.Once internetarchive/infogami#221 is merged, author name resolution will be case sensitive again. However, we can't simply update the Open Library code in
openlibrary/catalog/add_book/load_book.py
to use~i
, because of the performance issues associated with theILIKE
query, so we'll need to investigate further (perhaps usingEXPLAIN
can help us see more about the query.Proposal & Constraints
What is the proposed solution / implementation?
None yet -- this will take more investigation to figure out why
ILIKE
was such significant performance issues.Leads
Related files
Stakeholders
Note: Before making a new branch or updating an existing one, please ensure your branch is up to date.
The text was updated successfully, but these errors were encountered: