-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61281/
-----------------------------------------------------------
(Updated Aug. 2, 2017, 9:12 p.m.)
Review request for geode, anilkumar gingade, Darrel Schneider, Lynn Gallinat,
and Nick Reich.
Changes
-------
Fix review comments and handle distributed tx.
Bugs: GEODE-3379
https://issues.apache.org/jira/browse/GEODE-3379
Repository: geode
Description
-------
Current geode commit method only hold primary bucket lock when the txRegion is
a primary bucket. However, in between the tx operation and commit, a rebalance
could cause primary bucket to move. In this case, commit should detect the
primary bucket has been moved and throw TransactionDataRebalancedException.
Diffs (updated)
-----
geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java 2c8c28b
geode-core/src/test/java/org/apache/geode/disttx/PRDistTXDUnitTest.java
77bf740
geode-core/src/test/java/org/apache/geode/disttx/PRDistTXWithVersionsDUnitTest.java
9ab5145
geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRTransactionDUnitTest.java
6578baa
Diff: https://reviews.apache.org/r/61281/diff/2/
Changes: https://reviews.apache.org/r/61281/diff/1-2/
Testing
-------
Ongoing precheckin.
Thanks,
Eric Shu
