Skip to content

Remove unused ia_count Solr field (ref #11586)#11642

Merged
cdrini merged 2 commits intointernetarchive:masterfrom
bettercallok:remove-ia-count
Jan 2, 2026
Merged

Remove unused ia_count Solr field (ref #11586)#11642
cdrini merged 2 commits intointernetarchive:masterfrom
bettercallok:remove-ia-count

Conversation

@bettercallok
Copy link
Contributor

Description

Removes the unused ia_count Solr field as part of the cleanup effort in #11586.

Changes

  • Removed ia_count field definition from conf/solr/conf/managed-schema.xml
  • Removed ia_count type from openlibrary/solr/solr_types.py
  • Removed ia_count from all_fields in openlibrary/plugins/worksearch/schemes/works.py
  • Removed the scans sort option which depended on this field

Part of #11586

Copy link
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, thank you @bettercallok ! Not testing, low-risk refactor.

@cdrini cdrini self-assigned this Jan 2, 2026
@cdrini cdrini merged commit 5b59f76 into internetarchive:master Jan 2, 2026
4 checks passed
mystic-06 pushed a commit to mystic-06/openlibrary that referenced this pull request Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants