Merge pull request #8822 from internetarchive/fix-solr-reindex

Make osp-dump optional argument
internetarchive · Feb 24, 2024 · a30c9ac · a30c9ac
2 parents 07b33c5 + 4012c1f
commit a30c9ac
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/openlibrary/solr/update.py b/openlibrary/solr/update.py
@@ -155,7 +155,7 @@ def load_configs(
 
 async def main(
     keys: list[str],
-    osp_dump: Path,
+    osp_dump: Path | None = None,
     ol_url="http://openlibrary.org",
     ol_config="openlibrary.yml",
     output_file: str | None = None,

diff --git a/openlibrary/utils/open_syllabus_project.py b/openlibrary/utils/open_syllabus_project.py
@@ -18,7 +18,7 @@ def get_osp_dump_location() -> Path | None:
     return osp_dump_location
 
 
-def set_osp_dump_location(val: Path):
+def set_osp_dump_location(val: Path | None):
     global osp_dump_location
     osp_dump_location = val
 

diff --git a/scripts/solr_builder/solr_builder/solr_builder.py b/scripts/solr_builder/solr_builder/solr_builder.py
@@ -378,7 +378,7 @@ def build_job_query(
 async def main(
     cmd: Literal['index', 'fetch-end'],
     job: Literal['works', 'orphans', 'authors', 'lists'],
-    osp_dump: Path,
+    osp_dump: Path | None = None,
     postgres="postgres.ini",
     ol="http://ol/",
     ol_config="../../conf/openlibrary.yml",

diff --git a/scripts/solr_updater.py b/scripts/solr_updater.py
@@ -244,7 +244,7 @@ async def update_keys(keys):
 
 async def main(
     ol_config: str,
-    osp_dump: Path,
+    osp_dump: Path | None = None,
     debugger: bool = False,
     state_file: str = 'solr-update.state',
     exclude_edits_containing: str | None = None,