-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37450/
-----------------------------------------------------------
Review request for geode and Dan Smith.
Bugs: GEODE-216
https://issues.apache.org/jira/browse/GEODE-216
Repository: geode
Description
-------
The old code was trying to prevent a deadlock under some certain
conditions. But if it can not get a PR because of an exception
then a deadlock is not possible. So the new code just catches
and ignores exceptions and falls through to the code that will
process the message the old way. This code will encounter the
same exception and has the proper logic to handle it.
Diffs
-----
gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/partitioned/GetMessage.java
c8d990d7e2dba1f61a2aa7f9df703c0be6306a9d
Diff: https://reviews.apache.org/r/37450/diff/
Testing
-------
RedisDistDUnitTest was failing for me without this change.
With it testConcCreateDestroy has been passing every time.
Thanks,
Darrel Schneider