Skip to content

Commit ad730f0

Browse files
committed
Remove unused ia_loaded_id Solr field (ref #11586)
1 parent 65a311c commit ad730f0

File tree

3 files changed

+2
-22
lines changed

3 files changed

+2
-22
lines changed

conf/solr/conf/managed-schema.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@
155155
<field name="lccn" type="text_general" multiValued="true"/>
156156
<field name="ia" type="string" multiValued="true"/>
157157
<field name="ia_box_id" type="string" multiValued="true"/>
158-
<field name="ia_loaded_id" type="string" multiValued="true"/>
158+
159159
<field name="ia_count" type="pint"/>
160160
<field name="ia_collection" type="string" multiValued="true" />
161161
<field name="oclc" type="text_en_splitting" multiValued="true"/>

openlibrary/solr/solr_types.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class SolrDocument(TypedDict):
3232
lccn: Optional[list[str]]
3333
ia: Optional[list[str]]
3434
ia_box_id: Optional[list[str]]
35-
ia_loaded_id: Optional[list[str]]
35+
3636
ia_count: Optional[int]
3737
ia_collection: Optional[list[str]]
3838
oclc: Optional[list[str]]

openlibrary/solr/updater/work.py

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -608,27 +608,9 @@ def language(self) -> set[str]:
608608
return {lang for ed in self._solr_editions for lang in ed.language}
609609

610610
def build_legacy_ia_fields(self) -> dict:
611-
ia_loaded_id = set()
612611
ia_box_id = set()
613612

614613
for e in self._editions:
615-
# When do we write these to the actual edition?? This code might
616-
# be dead.
617-
if e.get('ia_loaded_id'):
618-
if isinstance(e['ia_loaded_id'], str):
619-
ia_loaded_id.add(e['ia_loaded_id'])
620-
else:
621-
try:
622-
assert isinstance(e['ia_loaded_id'], list)
623-
assert isinstance(e['ia_loaded_id'][0], str)
624-
except AssertionError:
625-
logger.error(
626-
"AssertionError: ia=%s, ia_loaded_id=%s",
627-
e.get("ia"),
628-
e['ia_loaded_id'],
629-
)
630-
raise
631-
ia_loaded_id.update(e['ia_loaded_id'])
632614
if e.get('ia_box_id'):
633615
if isinstance(e['ia_box_id'], str):
634616
ia_box_id.add(e['ia_box_id'])
@@ -643,8 +625,6 @@ def build_legacy_ia_fields(self) -> dict:
643625

644626
doc = {}
645627

646-
if ia_loaded_id:
647-
doc['ia_loaded_id'] = list(ia_loaded_id)
648628
if ia_box_id:
649629
doc['ia_box_id'] = list(ia_box_id)
650630
return doc

0 commit comments

Comments
 (0)