highperfocused/danswer
1
0
Fork 0
mirror of https://github.com/danswer-ai/danswer.git synced 2025-09-19 20:24:32 +02:00
Code Issues Packages Projects Releases Wiki Activity

Vespa edge case ID does not follow expected format (#541)

Browse Source
This commit is contained in:
Yuhong Sun
2023-10-08 13:36:00 -07:00
committed by GitHub
parent e8d3190770
commit 9b185f469f
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
backend/danswer/datastores/vespa/store.py
View File

@@ -104,7 +104,13 @@ def _get_vespa_chunk_ids_by_document_id(
while True:
results = requests.get(SEARCH_ENDPOINT, params=params).json()
hits = results["root"].get("children", [])
doc_chunk_ids.extend([hit["id"].split("::")[1] for hit in hits])
# Temporary logging to catch the rare index out of bounds issue
problematic_ids = [hit["id"] for hit in hits if len(hit["id"].split("::")) < 2]
if problematic_ids:
logger.error(f'IDs without "::" {problematic_ids}')
doc_chunk_ids.extend([hit["id"].split("::", 1)[-1] for hit in hits])
params["offset"] += hits_per_page # type: ignore
if len(hits) < hits_per_page: