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 heartbeat goroutine leak #149

Closed
wants to merge 5 commits into from
Closed

Fix heartbeat goroutine leak #149

wants to merge 5 commits into from

Commits on Jul 7, 2023

  1. 1. fix heartbeat go-routine leak. 

    2. update cleaner test.
    git committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    246a55e View commit details
    Browse the repository at this point in the history

  2. add comments

    git committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    b839c9f View commit details
    Browse the repository at this point in the history

  3. set connection.heartbeatStop to nil when exit heartbeat in case of bo… 

    …th nomal stop and reach error limits.
    git committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    c7f4ada View commit details
    Browse the repository at this point in the history

  4. stop heartbeat even there are no open queues

    git committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    a4ec6f4 View commit details
    Browse the repository at this point in the history

  5. rename StopAllConsuming to Close. 

    This method could be called by consumer or producer.
    git committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    007c875 View commit details
    Browse the repository at this point in the history