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

when DefaultMQProducerImpl shutdown didn't shutdown some demon thread. #1950

Closed
lilylei013 opened this issue Apr 20, 2020 · 5 comments · Fixed by #1958
Closed

when DefaultMQProducerImpl shutdown didn't shutdown some demon thread. #1950

lilylei013 opened this issue Apr 20, 2020 · 5 comments · Fixed by #1958

Comments

@lilylei013
Copy link

BUG REPORT

  1. Please describe the issue you observed:
  • What did you do (The steps to reproduce)?
    start a DefaultMQProducer producer , after send message , then shutdown it, but the main thread is not finish.

  • What did you expect to see?
    I want all the MQ producer relative threads is finished.

  • What did you see instead?
    there is still some demon threads left there .
    RequestHouseKeepingService , and AsyncAppender-Dispatcher-Thread-3,4,6

  1. Please tell us about your environment:
    windows 64bit eclipse
  2. Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):
    for the RequestHouseKeepingService , there is because timer is not shutdown
    for the AsyncAppender-Dispatcher-Thread , I didn't found it now
@RongtongJin
Copy link
Contributor

Hi @lilylei013 What version are you using? I used the latest version and there seems to be no such issue.

@lilylei013
Copy link
Author

I checked 4.7.0 version, it's not fix. It's the timertask which is not cancel

@coder-zzzz
Copy link
Contributor

I found this issue #1957 today, but didn't find this issue #1950.I already commit a pr #1958 to fix #1957

@coder-zzzz
Copy link
Contributor

I think the AsyncAppender-Dispatcher-Thread is used by log, so it is usefull , can not shutdown when producer shutdown

@lilylei013
Copy link
Author

Just cancle the timer when producer shutdown it's okay

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

Successfully merging a pull request may close this issue.

3 participants