----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47543/ -----------------------------------------------------------
(Updated May 26, 2016, 10:44 p.m.) Review request for geode, Darrel Schneider and Swapnil Bawaskar. Changes ------- Make TXManagerImpl mockable so it can be mocked in the unit test. Only unmasqerade when TXStateProxy is masqeraded. Avoid to use Thread.sleep() in the unit test. Repository: geode Description ------- This also handles when an inflight msg arrived after the transaction is completed after a failover. Diffs (updated) ----- geode-core/src/main/java/com/gemstone/gemfire/internal/cache/RemoteOperationMessage.java 42ce811 geode-core/src/main/java/com/gemstone/gemfire/internal/cache/TXManagerImpl.java 4b2f904 geode-core/src/main/java/com/gemstone/gemfire/internal/cache/partitioned/PartitionMessage.java 626efef geode-core/src/test/java/com/gemstone/gemfire/internal/cache/RemoteOperationMessageTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/TXManagerImplTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PartitionMessageTest.java PRE-CREATION Diff: https://reviews.apache.org/r/47543/diff/ Testing ------- precheckin Thanks, Eric Shu
