-
Notifications
You must be signed in to change notification settings - Fork 629
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-5477 Too agressive unboxing leading to wrong L1 returns #3476
Conversation
CompatibilityCacheFactory.killCacheFactories(cacheFactory1, cacheFactory2); | ||
} | ||
|
||
// public void testHotRodPutInNonOwner() { |
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.
Are we adding this back in? If not we can remove the SplitMagicKey class it seems.
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.
There seems to be an overlap in functionality between getSplitIntKeyForServer
and SplitMagicKey
anyway...
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.
Leftover stuff, I'll clean it up and update.
fb177d7
to
cdced1a
Compare
* Unboxing of entries for external client consumption should only happen when commands are local, since remote commands come from internal clustering.
Updated |
Integrated, thanks Galder! |
And now in 7.2.x as well. |
@danberindei @galderz I'm not sure what the problem is but it seems these changes cause the tests to hang on master. Reverting them fixes the problem. |
https://issues.jboss.org/browse/ISPN-5477