-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52172/
-----------------------------------------------------------
Review request for geode, Hitesh Khamesra and Udo Kohlmeyer.
Bugs: GEODE-1927
https://issues.apache.org/jira/browse/GEODE-1927
Repository: geode
Description
-------
This adds a check in InternalDataSerializer for com.gemstone.gemfire packages
and transforms them to org.apache.geode.
I've also added a hook for translating org.apache.geode exceptions into
com.gemstone.gemfire exceptions. We've decided to keep com.gemstone.gemfire
exceptions out of the Geode repository to avoid the confusion it would cause to
have an org.apache.geode implementation throw com.gemstone.gemfire exceptions.
The latter change required changes to some tests using mocks to represent
ServerConnections. These mocks were returning null from getClientVersion(),
causing NPEs in client/server code.
Diffs
-----
geode-core/src/main/java/org/apache/geode/internal/InternalDataSerializer.java
6e7aa55bd8a8b99c63f39965daa0222f42f64d30
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/BaseCommand.java
9a78772c466996f83383f4e63cb3d1d6654172a0
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/OldClientSupport.java
PRE-CREATION
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/ContainsKey66Test.java
6728a377816e521a9419e8dd7576397f8548c14e
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/CreateRegionTest.java
389399191a3a69a1f591d029ecf3ea3981f845db
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/Destroy65Test.java
ed76cb6a965208b315ebb004cef39ea763d4b686
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/DestroyRegionTest.java
49a95a0bdae5f4fb51c0d6fcd8d5513805b45d26
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/DestroyTest.java
422733e4f5e958add74b4b7586bb123703776bc7
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/Get70Test.java
4b63a072370746e950ef3697358c7be7a125a213
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/InvalidateTest.java
2dcdd0413c292acd3a9ef40f9809f23f5e95ca29
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/Put61Test.java
c368ba89435303f498a212defdc2508426366093
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/Put65Test.java
830884c858ae3c6fd4121ed60343e4195b55d77a
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/PutTest.java
a9c3af43de1b5b23a958d22b577091e6ccf0bca5
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/RequestTest.java
b6997bdef1b51e10780caf1eee6e3a7c46424f98
geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/command/UnregisterInterestTest.java
2da6f19a2708b839264c14b9c27e3ff6702a2b52
geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ProcessManager.java
1c87b1a0346445da95c9d510ecc8b8e6e996bac2
Diff: https://reviews.apache.org/r/52172/diff/
Testing
-------
precheckin. New tests are being developed for gemfire<->geode WAN and
client/server interactions but these won't be part of the Geode repo.
Thanks,
Bruce Schuchardt
