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
ISPN-12548 Replicated cache get ignores value in zero-capacity nodes #9024
ISPN-12548 Replicated cache get ignores value in zero-capacity nodes #9024
Conversation
* Do not use GetReplCache on zero-capacity nodes * Extend ClusteredLockWithZeroCapacityNodesTest * Remove REPL_SYNC references in clustered locks tests * Make waitForNoRebalance work with zero-capacity nodes
* Split TestFunctionalWriteOperation from TestWriteOperation. * Make operation REPLACE_META_FUNCTIONAL work outside NonTxBackupOwnerBecomingPrimaryOwnerTest. * Extend ZeroCapacityNodeTest to run all functional and non-functional write operations.
…d caches * Keep broadcasting single-key commands to all the members, but to not wrap any key and do not perform the write. * Since many-key commands use unicasts, don't send them to zero-capacity nodes at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one minor nitpick. @danberindei Can you backport to 12.0.x so that we can get this in the operator?
* Represents a functional write operation to test. | ||
* | ||
* @author Dan Berindei | ||
* @since 12.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
12.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ryanemerson if I backport it to 12.0.1, then wouldn't the @since 12.0
tag be correct? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
touché! 😆
@ryanemerson backport open: #9034 |
Thanks @danberindei |
Also ISPN-12662 Extend zero-capacity node testing in core
https://issues.redhat.com/browse/ISPN-12548
https://issues.redhat.com/browse/ISPN-12662
Do not use GetReplCache on zero-capacity nodes
Extend ClusteredLockWithZeroCapacityNodesTest
Remove REPL_SYNC references in clustered locks tests
Make waitForNoRebalance work with zero-capacity nodes
Split TestFunctionalWriteOperation from TestWriteOperation.
Make operation REPLACE_META_FUNCTIONAL work outside
NonTxBackupOwnerBecomingPrimaryOwnerTest.
Extend ZeroCapacityNodeTest to run all functional and non-functional
write operations.
Keep broadcasting single-key commands to all the members,
but to not wrap any key and do not perform the write.
Since many-key commands use unicasts, don't send them to
zero-capacity nodes at all.