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.

Sign up for GitHub

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

Jump to bottom

Online status heartbeat renewal #1592

Open
wang-qijia opened this issue Dec 12, 2024 · 0 comments
Open

Online status heartbeat renewal #1592

wang-qijia opened this issue Dec 12, 2024 · 0 comments
Labels
question How to do something

Comments

@wang-qijia
Copy link

"Thank you for contributing to this high-quality chat project. I encountered an issue during my review and would like to discuss it.

In the HeartbeatManager#updateOnlineUsersTtl method, the estimatedUserCountToRefreshPerInterval parameter does not seem to limit the Redis user count as expected. Is this a known issue in the current implementation, or is it intended behavior?"
@JamesChenX JamesChenX added the question How to do something label Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question How to do something
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wang-qijia @JamesChenX