----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46917/ -----------------------------------------------------------
(Updated May 3, 2016, 1:03 a.m.) Review request for geode, anilkumar gingade and nabarun nag. Changes ------- Appying review comments from Jason, Naba - using ExpectedException in the tests instead of @Test(expected = ...) Repository: geode Description ------- Testing how functions propagate exceptions from within the function to the caller or the result collector. Fixing a bug in exception propegation discovered while writing these tests. Diffs (updated) ----- geode-core/src/main/java/com/gemstone/gemfire/internal/cache/PartitionedRegionDataStore.java 57b1e71e8a5591e74f11041c916ed5cec1a8fd29 geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServiceBase.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServiceLocalPRDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServiceLocalRRDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServiceMultipleOnMemberDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServicePeerAccessorPRDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServicePeerAccessorRRDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/FunctionServiceSingleOnMemberDUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/46917/diff/ Testing ------- Thanks, Dan Smith
