Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Update) Use meilisearch for torrent search by default #4236

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

Roardom
Copy link
Contributor

@Roardom Roardom commented Oct 17, 2024

Using scout directly doesn't work because when increasing the maxTotalHits in meilisearch, scout will use the max instead of just fetching the current page. Meilisearch seems to handle a max of 200k torrents fine on its own, with 12ms latency when e.g. filtering by type down to 38000 totalHits. Run php artisan scout:sync-index-settings, followed by php artisan auto:sync_torrents_to_meilisearch after updating.

Using scout directly doesn't work because when increasing the maxTotalHits in meilisearch, scout will use the max instead of just fetching the current page. Meilisearch seems to handle a max of 200k torrents fine on its own, with 12ms latency when e.g. filtering by type down to 38000 totalHits. Run `php artisan scout:sync-index-settings`, followed by `php artisan auto:sync_torrents_to_meilisearch` after updating.
@Roardom Roardom force-pushed the meilisearch-torrent-search branch from 469a09f to 5b7bf9f Compare October 17, 2024 07:21
@HDVinnie HDVinnie merged commit ad2b942 into HDInnovations:8.x.x Oct 17, 2024
5 checks passed
@HDVinnie HDVinnie deleted the meilisearch-torrent-search branch October 17, 2024 12:01
Roardom added a commit to Roardom/UNIT3D-Community-Edition that referenced this pull request Oct 18, 2024
We need to sort the torrents after they've been fetched from the database to match the same order that meilisearch returned.

Fixes regression from HDInnovations#4236.
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.

2 participants