----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56333/#review164444 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/internal/cache/tx/ClientTXStateStub.java (line 82) <https://reviews.apache.org/r/56333/#comment236167> how about changing it to "txRolledback" to indicate tx is not valid anymore. geode-core/src/main/java/org/apache/geode/internal/cache/tx/ClientTXStateStub.java (line 213) <https://reviews.apache.org/r/56333/#comment236168> Add unit test... - anilkumar gingade On Feb. 6, 2017, 5:45 p.m., Eric Shu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56333/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2017, 5:45 p.m.) > > > Review request for geode, anilkumar gingade and Darrel Schneider. > > > Bugs: GEODE-2429 > https://issues.apache.org/jira/browse/GEODE-2429 > > > Repository: geode > > > Description > ------- > > Do not run JTA afterCompletion on server if beforeCompletion is failed. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/cache/tx/ClientTXStateStub.java > e79324d > > Diff: https://reviews.apache.org/r/56333/diff/ > > > Testing > ------- > > precheckin. > > > Thanks, > > Eric Shu > >