Skip to content

fix(scrapeURL/index): locations array querying bug#2164

Merged
mogery merged 1 commit intomainfrom
mogery/eng-3565-fix-index-locations-array-querying-bug
Sep 16, 2025
Merged

fix(scrapeURL/index): locations array querying bug#2164
mogery merged 1 commit intomainfrom
mogery/eng-3565-fix-index-locations-array-querying-bug

Conversation

@mogery
Copy link
Copy Markdown
Member

@mogery mogery commented Sep 16, 2025

Summary by cubic

Fixes index lookups that used the exact order of location.languages, causing cache misses and wrong matches. Now we use the index_get_recent RPC to handle language sets and nulls consistently. Addresses ENG-3565.

  • Bug Fixes
    • Treats location.languages as a set; order no longer affects index/cache hits.
    • Converts empty languages arrays to null for correct matching.
    • Updates tests to verify order-insensitive behavior.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@mogery mogery merged commit 8f4b73b into main Sep 16, 2025
4 of 13 checks passed
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.

1 participant