----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51188/#review146033 -----------------------------------------------------------
Ship it! Ship It! - Barry Oglesby On Aug. 17, 2016, 5:41 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51188/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2016, 5:41 p.m.) > > > Review request for geode and Barry Oglesby. > > > Repository: geode > > > Description > ------- > > Read all of the results from the wire in > ExecuteRegionFunctionOp.processResponse, even if we read an exception as > one of the responses. This ensures that all data is read off the wire. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/ExecuteFunctionOp.java > 278f1f79afa98823df27974da2907685c7e5e641 > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/ExecuteRegionFunctionOp.java > d1540204546604ff7ed3d3d490ac40d079ac2953 > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/ExecuteRegionFunctionSingleHopOp.java > ce7c3f84092f59051e5a148a42cc6f06adc787fe > > Diff: https://reviews.apache.org/r/51188/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >