----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43502/#review119392 -----------------------------------------------------------
Ship it! Ship It! - anilkumar gingade On Feb. 12, 2016, 6:18 p.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43502/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2016, 6:18 p.m.) > > > Review request for geode, anilkumar gingade, Barry Oglesby, and Dan Smith. > > > Repository: geode > > > Description > ------- > > The error handling for functions is different depending on if it failed > locally or remotely. In this case, the function was failing on the remote > node and the try/catch block would have it send the exception, however there > are two types of send exception calls. We needed it to use the result > sender's way to send the exception... > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/command/ExecuteRegionFunction66.java > 6d9ca49 > > Diff: https://reviews.apache.org/r/43502/diff/ > > > Testing > ------- > > Running precheckin. > The failing test ran 10000 times without failure. Previously it would fail 1 > in 5 if not more often. > > > Thanks, > > Jason Huynh > >
