Skip to content

make FulltextSearchSuggestion partial async#12265

Merged
RayBB merged 3 commits intointernetarchive:masterfrom
Sanket17052006:async-fulltextsearch-12257
Apr 2, 2026
Merged

make FulltextSearchSuggestion partial async#12265
RayBB merged 3 commits intointernetarchive:masterfrom
Sanket17052006:async-fulltextsearch-12257

Conversation

@Sanket17052006
Copy link
Copy Markdown
Contributor

@Sanket17052006 Sanket17052006 commented Apr 2, 2026

Closes #12257

Changes :

  • Makes the FulltextSearchSuggestion partial asynchronous.
  • Added test_fulltext_search_suggestion_partial.py

Testing

  • docker compose run --rm home pytest openlibrary/tests/fastapi/test_fulltext_search_suggestion_partial.py -v

Screenshot

N/A

Stakeholders

@RayBB

Copy link
Copy Markdown
Collaborator

@RayBB RayBB left a comment

Choose a reason for hiding this comment

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

Excellent work here, this is very good. I'm going to remove the test file because we don't really need that for these endpoints and put it on testing to double check it.

@RayBB
Copy link
Copy Markdown
Collaborator

RayBB commented Apr 2, 2026

It's working excellent and should be a big help to our performance issues. Thanks for doing this one!

@RayBB RayBB merged commit 6ec93d5 into internetarchive:master Apr 2, 2026
3 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.

🟢 /partials/FulltextSearchSuggestion.js

2 participants