-----------------------------------------------------------
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
> 
>

Reply via email to