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

Unclear docs for TTL expiration #11787

Closed
balazspeterfi opened this issue Nov 14, 2017 · 0 comments
Closed

Unclear docs for TTL expiration #11787

balazspeterfi opened this issue Nov 14, 2017 · 0 comments

Comments

@balazspeterfi
Copy link

@balazspeterfi balazspeterfi commented Nov 14, 2017

Since the new eviction algorithm has been introduced (v3.7) the documentation suggests that the expiration based eviction only happens as part of the eviction sampling, which only occurs when the cache reaches its maximum size. I realised that it's not the case and in fact expiration happens independently from eviction.

The misleading points are:

  • time-to-live-seconds and max-idle-seconds are explained under Map Eviction Configuration, and it doesn't say when these expirations get evaluated
  • in the Eviction Algorithm section in step 6 it says:
  1. The eviction strategy tests whether the sampled entry is already expired (lazy expiration). If expired, the sampling stops and the entry is removed in step 9.

the fact that expiration based eviction is only mentioned here and it's referred as "lazy expiration" gives the false impression that it is the only point when expiration is being checked.

Could you please clarify this in the docs to give a clearer view on how expiration is working?

@jerrinot jerrinot added this to the 3.10 milestone Nov 14, 2017
@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 19, 2018
taburet added a commit to taburet/hazelcast that referenced this issue Feb 20, 2018
taburet added a commit to taburet/hazelcast that referenced this issue Feb 20, 2018
taburet added a commit that referenced this issue Feb 26, 2018
Fixes: #11787
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.

5 participants