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.

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

CachePutBackupOperation causes LocalCacheStats.eviction count increase on backup owner #12931

Closed
gokhanoner opened this issue Apr 20, 2018 · 0 comments · Fixed by #12976
Closed

CachePutBackupOperation causes LocalCacheStats.eviction count increase on backup owner #12931

gokhanoner opened this issue Apr 20, 2018 · 0 comments · Fixed by #12976
Labels
Module: ICache Source: Internal PR or issue was opened by an employee Type: Defect
Milestone
3.10.1

Comments

@gokhanoner
Copy link
Contributor

See https://github.com/hazelcast/hazelcast/blob/master/hazelcast/src/main/java/com/hazelcast/cache/impl/operation/CachePutBackupOperation.java#L72

When entry evicted on primary member, it also evicts on backup members. This causes LocalCacheStats.getCacheEvictions return wrong numbers.

Please see below ManCenter screenshot. Even though only 1 entry evicted, see put count & entry count, it seems 2 entry evicted.

screen shot 2018-04-19 at 9 46 25 pm
@gokhanoner gokhanoner changed the title CachePutBackupOperation causes eviction stat increase on backup member LocalCacheStats CachePutBackupOperation causes LocalCacheStats.eviction count increase on backup owner Apr 20, 2018
@vbekiaris vbekiaris mentioned this issue Apr 25, 2018
Merged
@vbekiaris vbekiaris added this to the 3.10.1 milestone May 3, 2018
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Module: ICache Source: Internal PR or issue was opened by an employee Type: Defect
Projects
None yet
Milestone
3.10.1
Development

Successfully merging a pull request may close this issue.

Only increment cache eviction stats on primary record store vbekiaris/hazelcast
3 participants
@vbekiaris @gokhanoner @mmedenjak