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

Update Hazelcast.shutdownAll javadoc #12012

Closed
kwart opened this issue Dec 15, 2017 · 0 comments
Closed

Update Hazelcast.shutdownAll javadoc #12012

kwart opened this issue Dec 15, 2017 · 0 comments

Comments

@kwart
Copy link
Contributor

@kwart kwart commented Dec 15, 2017

The Hazelcast.shutdownAll() JavaDoc says:

Shuts down all running Hazelcast Instances on this JVM.

It's not true, as the native clients are also HazelcastInstance instances and they are not shut down when the method is called.

Suggested fix

  • either improve the JavaDoc to sth like: Shuts down all running member Hazelcast Instances on this JVM.
  • or change the method behavior to shut down the native clients too
@kwart kwart added the Team: Core label Dec 15, 2017
@mmedenjak mmedenjak added this to the 3.10 milestone Jan 15, 2018
@taburet taburet added Team: Core and removed Team: Core labels Jan 22, 2018
@Holmistr Holmistr added the DR3 label Feb 2, 2018
@taburet taburet self-assigned this Feb 8, 2018
taburet added a commit to taburet/hazelcast that referenced this issue Feb 19, 2018
taburet added a commit that referenced this issue Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

4 participants
You can’t perform that action at this time.