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

Fix runaway thread issue where MqttClient uses 100% CPU when disconnected #75

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix runaway thread issue where MqttClient uses 100% CPU when disconnected #75

wants to merge 4 commits into from

Commits on Dec 15, 2017

  1. Added tracelogging and fixes for race conditions related to closing f… 

    …ailed connections

Signed-off-by: Marcus Hellsten <marcus@thehellstens.com>
    @mhellsten @psmhellsten
    mhellsten authored and psmhellsten committed Dec 15, 2017
    Configuration menu
    Copy the full SHA
    f3b16fd View commit details
    Browse the repository at this point in the history

  2. Remove additional diagnostic trace logging to leave just the fix. 

    Signed-off-by: Marcus Hellsten <marcus@thehellstens.com>
    @mhellsten @psmhellsten
    mhellsten authored and psmhellsten committed Dec 15, 2017
    Configuration menu
    Copy the full SHA
    7110f6b View commit details
    Browse the repository at this point in the history

  3. Made change to force github to repull updated history using corrected… 

    … email with accepted ECA.

Signed-off-by: Marcus Hellsten <marcus@thehellstens.com>
    @mhellsten
    mhellsten committed Dec 15, 2017
    Configuration menu
    Copy the full SHA
    8e6625a View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2018

  1. Bug fix: When all broker instances are down, but we have a load balan… 

    …cer in front of them, the connection may be accepted even though nobody is listening. In that situation, when reconnecting, we may end up waiting indefinitely for the KeepAlive thread to exit, when in fact it already stopped as part of the original disconnection event.
    @psmhellsten
    psmhellsten committed Jan 17, 2018
    Configuration menu
    Copy the full SHA
    2ffb689 View commit details
    Browse the repository at this point in the history