Skip to content

fix: 504 on gameplay-format import confirm — batch the Meilisearch re… - #67

Merged
toxicity1985 merged 1 commit into
mainfrom
gameplay-format-implementation
Jul 10, 2026
Merged

fix: 504 on gameplay-format import confirm — batch the Meilisearch re…#67
toxicity1985 merged 1 commit into
mainfrom
gameplay-format-implementation

Conversation

@toxicity1985

Copy link
Copy Markdown
Collaborator

…sync

Flushing hundreds of updated CardGroups fired MeilisearchSyncListener once per group, each reindexing every Card in that group with its own synchronous HTTP call — thousands of round-trips in a single request. Disable the per-entity listeners during the bulk write (same pattern already used by ImportEquinoxCardsCommand) and resync the affected groups afterwards in one batched Meilisearch call.

…sync

Flushing hundreds of updated CardGroups fired MeilisearchSyncListener once
per group, each reindexing every Card in that group with its own
synchronous HTTP call — thousands of round-trips in a single request.
Disable the per-entity listeners during the bulk write (same pattern
already used by ImportEquinoxCardsCommand) and resync the affected
groups afterwards in one batched Meilisearch call.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@toxicity1985
toxicity1985 merged commit a6429c2 into main Jul 10, 2026
2 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