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

Added EntryExpiredListener in order to listen expired map entries. #6310

Conversation

ahmetmircik
Copy link
Member

closes #6311

backport of #6308

  • Created an EntryExpiredListener which extends MapListener.
  • Kept current EntryEvictedListener notification mechanism as is; means it will be receiving both expiration and eviction events as in now. If one implements both EntryExpiredListener and EntryEvictedListener interfaces, there is a probability that more than one event per entry at the same time may be seen. So you may receive both EVICTED and EXPIRED events for the same entry. This is because, eviction may also sweep expired entries. This may be appear when both size-based-eviction and expiration are configured for the same IMap.

@ahmetmircik ahmetmircik added this to the 3.5.3 milestone Sep 29, 2015
@ahmetmircik ahmetmircik force-pushed the feature/3.5.3/entryExpiredListener branch from 53e65de to 3d2b219 Compare September 29, 2015 12:48
@ahmetmircik ahmetmircik force-pushed the feature/3.5.3/entryExpiredListener branch from 3d2b219 to a427e28 Compare September 29, 2015 12:49
@ahmetmircik
Copy link
Member Author

This PR seems not applicable to maintenance. If we merge this PR, in a cluster which has both 3.5.3 and 3.5.2 nodes, 3.5.2 nodes will throw lots of NPEs due to the received EXPIRED events. The code part that throws NPE is here.

Closing this PR.

@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Source: Internal PR or issue was opened by an employee Team: Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants