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

CacheRemoveAllOperation doesn't send completed event in some cases #5865

Closed
serkan-ozal opened this issue Aug 5, 2015 · 0 comments · Fixed by #6029
Closed

CacheRemoveAllOperation doesn't send completed event in some cases #5865

serkan-ozal opened this issue Aug 5, 2015 · 0 comments · Fixed by #6029
Assignees
@serkan-ozal
Labels
[OLD]Team: Integration Source: Internal PR or issue was opened by an employee Type: Defect
Milestone
3.6

Comments

@serkan-ozal
Copy link
Contributor

CacheRemoveAllOperation doesn't send completed event

  • If CacheRecordStore for that partition is not created yet
  • If the filtered keys are empty (that partition is not owner of the any key to be removed)

The solution is sending completed event in these cases.

This failure can be reproduced by getting cache from the second node instead of first node in BasicCacheTest::testCompletionTest. I have tried the fix idea above on my local and it worked.
@serkan-ozal serkan-ozal self-assigned this Aug 5, 2015
@serkan-ozal serkan-ozal added this to the 3.6 milestone Aug 5, 2015
serkan-ozal pushed a commit to serkan-ozal/hazelcast that referenced this issue Aug 20, 2015
Be sure that completion event is sent in all cases for `CacheRemoveAl…
e7e119c 
…lOperation`. Fixes hazelcast#5865
@serkan-ozal serkan-ozal mentioned this issue Aug 20, 2015
Merged
serkan-ozal pushed a commit to serkan-ozal/hazelcast that referenced this issue Aug 20, 2015
Be sure that completion event is sent in all cases for `CacheRemoveAl…
d3cbff0 
…lOperation`. Fixes hazelcast#5865
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Aug 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@serkan-ozal serkan-ozal

Labels
[OLD]Team: Integration Source: Internal PR or issue was opened by an employee Type: Defect
Projects
None yet
Milestone
3.6
2 participants
@serkan-ozal @mmedenjak