Near cache consistency improvements #9660
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ahmetmircik
changed the title
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
3 times, most recently
from
4949926
to
85ce61b
Compare
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
2 times, most recently
from
a54515f
to
bbfa48d
Compare
metanet
approved these changes
Jan 30, 2017
jerrinot
approved these changes
Jan 31, 2017
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
12b137b
to
0a477ac
Compare
|
@ahmetmircik: what's preventing to have this merged? |
|
@jerrinot ee counterpart is not ready yet. |
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
2 times, most recently
from
826053e
to
22adb93
Compare
|
Fixes #9536 |
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
3 times, most recently
from
b7ffc81
to
3f3b5de
Compare
|
Test PASSed. |
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
3f3b5de
to
071f18b
Compare
|
Test PASSed. |
ahmetmircik
changed the title
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
071f18b
to
76bff1a
Compare
|
Test PASSed. |
vbekiaris
approved these changes
Feb 7, 2017
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
76bff1a
to
02bb976
Compare
|
@vbekiaris fixed |
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
02bb976
to
952a912
Compare
|
Test PASSed. |
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
952a912
to
f112a07
Compare
- Removed stale read probability during near cache updates. - Removed `KeyStateMarker` and introduced record states. A record can have 2 main states now RESERVED, READ_PERMITTED. This new approach will help to utilize near cache more effectively compared to existing approach. Because in existing approach, only one thread can operate on a partition, if other threads want to put some other keys belonging to that partition, they can't do it because partition is already locked by first thread. - fixes hazelcast#9403 - Implemented `computeIfPresent` for `IConcurrentMap` and introduced `IBiFunction`
ahmetmircik
force-pushed
the
fix/3.8/cons2stDraft
branch
from
f112a07
to
0a00c1a
Compare
|
Test PASSed. |
1 similar comment
|
Test PASSed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
KeyStateMarkerand introduced record states. A record can have 3 states now UPDATE_STARTED, REMOVE_REQUESTED, READ_PERMITTED. This new approach will help to utilize near cache more effectively compared to existing approach. Because in existing approach, only one thread can operate on a partition, if other threads want to put some other keys belonging to that partition, they can't do it because partition is already locked by first thread.computeIfPresentandmergeforIConcurrentMapand introducedIBiFunctionPS: EE part will be introduced upon end of this PRs' review.