Skip to content

Conversation

@hero5512
Copy link
Contributor

No description provided.

@s1na s1na assigned s1na and zsfelfoldi and unassigned s1na Sep 24, 2025
@zsfelfoldi
Copy link
Contributor

Checking map indices being in the same group before doing the database read is not a meaningful optimization because this case should not happen ever, the check is just there to raise an error if a future change ever uses this function incorrectly.
Setting the exact capacity might be considered a slight optimization but the memory consumption during event search has never really been a bottleneck, and also the whole database implementation will soon be replaced by a more efficient one, therefore this change it not really worth merging,

@zsfelfoldi zsfelfoldi closed this Dec 6, 2025
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.

3 participants