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

FIX-898: initial commit and test fix #938

Conversation

AnthonyKot
Copy link

The initial attempt to display correctly number of unread messages and handle deleted message edge case.

@or-else
Copy link
Contributor

or-else commented Feb 17, 2025

Sorry, but I don't think it's an acceptable fix - it's too expensive. You are going to read potentially multiple large messages from the DB for each read receipt. That alone would likely generate DB load comparable to everything else combined, particularly if there is a lot of large group topics.

@AnthonyKot
Copy link
Author

AnthonyKot commented Feb 17, 2025

I agree, let's treat this as a draft solution. My plan is to verify it locally and then work on adding a new query to MongoDB to efficiently retrieve the count of unread messages.

@AnthonyKot AnthonyKot closed this Feb 19, 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.

2 participants