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

Improve the emqx_connection module using gen_statem behaviour #2235

Merged
merged 8 commits into from Feb 28, 2019

Conversation

@emqplus
Copy link
Contributor

emqplus commented Feb 19, 2019

No description provided.

Gilbert and others added 7 commits Jan 21, 2019
This chang the reason code to not authorized code.
* Fix bridge bug

* Fix ack bug

* Limit bridge QoS less than 1
Before this change, when shared dispatch ack is enabled (in config)
in case all subscribers are offline (all sessions gave negative ack)
the message is simply discarded.
In this change, it is ensured to have one session picked according to
configured dispatch strategy when no subscriber is online.
The messages dispatched in such scenario are then queued in session state.
@emqplus emqplus removed their assignment Feb 19, 2019
@turtleDeng turtleDeng changed the base branch from testing to develop Feb 19, 2019
@terry-xiaoyu terry-xiaoyu changed the base branch from develop to testing Feb 19, 2019
@terry-xiaoyu terry-xiaoyu changed the base branch from testing to develop Feb 19, 2019
@gilbertwong96 gilbertwong96 merged commit 08204fc into develop Feb 28, 2019
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
@turtleDeng turtleDeng deleted the channel-session-statem branch Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.