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

[BACKPORT] Remove outdated warnings regarding TTL from IMap's javadoc #12357

Merged

Conversation

@taburet
Copy link
Contributor

@taburet taburet commented Feb 15, 2018

There were outdated warnings in IMap javadoc regarding expiration and
TTL behavior of mutating methods don't accepting TTL as a parameter.

  1. Added the tests to verify the current behavior and to make sure we
    won't silently break it in the future one more time.

  2. Removed the warnings themselves.

  3. Added javadoc describing the current TTL behavior.

Fixes: #12144

There were outdated warnings in IMap javadoc regarding expiration and
TTL behavior of mutating methods don't accepting TTL as a parameter.

1. Added the tests to verify the current behavior and to make sure we
won't silently break it in the future one more time.

2. Removed the warnings themselves.

3. Added javadoc describing the current TTL behavior.

Fixes: #12144
@taburet taburet added the Team: Core label Feb 15, 2018
@taburet taburet self-assigned this Feb 15, 2018
@taburet taburet requested review from ahmetmircik and tkountis Feb 15, 2018
@tkountis tkountis changed the title Remove outdated warnings regarding TTL from IMap's javadoc [BACKPORT] Remove outdated warnings regarding TTL from IMap's javadoc Feb 15, 2018
@tkountis tkountis added this to the 3.9.4 milestone Feb 15, 2018
@taburet taburet merged commit 0e55122 into hazelcast:maintenance-3.x Feb 15, 2018
1 check passed
1 check passed
default Test PASSed.
Details
@taburet taburet deleted the taburet:fix/maintenance-3.x/issue-12144 branch Feb 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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