8000 Use an executor for shutdown monitors by michaelklishin · Pull Request #88 · rabbitmq/rabbitmq-java-client · GitHub
[go: up one dir, main page]
Skip to content

Use an executor for shutdown monitors #88

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
Merged
merged 4 commits into from
Sep 18, 2015
Merged

Use an executor for shutdown monitors #88

merged 4 commits into from
Sep 18, 2015

Conversation

michaelklishin
Copy link
Contributor

Fixes #87, references #86.

@michaelklishin michaelklishin self-assigned this Sep 18, 2015
@michaelklishin michaelklishin added this to the n/a milestone Sep 18, 2015
michaelklishin added a commit that referenced this pull request Sep 18, 2015
@michaelklishin
Merge pull request #88 from rabbitmq/rabbitmq-java-client-87
e762711 
Use an executor for shutdown monitors
@michaelklishin michaelklishin merged commit e762711 into master Sep 18, 2015
@YannRobert
Copy link
Contributor

@michaelklishin There is a remaining place where the shutdownExecutor should be used : com.rabbitmq.client.impl.ChannelManager#scheduleShutdownProcessing (this is the place I spotted in my original issue)

@michaelklishin
Copy link
Contributor Author

Filed #94 for that.

@acogoluegnes acogoluegnes deleted the rabbitmq-java-client-87 branch March 21, 2019 08:24
stream-iori pushed a commit to stream-iori/rabbitmq-java-client that referenced this pull request Mar 20, 2022
@HzjNeverStop
upgrade version to 1.3.0-SNAPSHOT (rabbitmq#88)
840d1f3 
* upgrade version to 1.3.0-SNAPSHOT

* upgrade version to 1.3.0-SNAPSHOT

Co-authored-by: 致节 <hzj266771@antgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@michaelklishin michaelklishin

Labels
None yet
Projects
None yet
Milestone
n/a
Development

Successfully merging this pull request may close these issues.
2 participants
@michaelklishin @YannRobert
0