Skip to content

feat(mnesia_tm): add async_dist switch #68

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
Open
wants to merge 1 commit into
base: emqx-OTP-26.2.5.2
Choose a base branch
from
Open

feat(mnesia_tm): add async_dist switch #68

wants to merge 1 commit into from

Conversation

qzhuyan
Copy link

@qzhuyan qzhuyan commented Apr 2, 2025 •
edited
Loading

@qzhuyan qzhuyan force-pushed the dev/william/mnesia_tm_async_dist_switch branch from 92123b2 to 5953e23 Compare April 2, 2025 10:19
@zmstone
Copy link
Member

zmstone commented Apr 2, 2025

do we have test results to address this concern?

from the otp doc

However, note that you need to make sure that flow control for data sent using fully asynchronous distributed signaling is implemented, or that the amount of such data is known to always be limited; otherwise, you may get into a situation with excessive memory usage.

@qzhuyan
Copy link
Author

qzhuyan commented Apr 3, 2025

I wrote the background in this ticket : https://emqx.atlassian.net/browse/EMQX-14067

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zmstone zmstone zmstone approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@qzhuyan @zmstone