----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61281/#review182124 -----------------------------------------------------------
Fix it, then Ship it! Ship It! geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java Lines 673 (patched) <https://reviews.apache.org/r/61281/#comment257975> For distTx we skip for taking locks on secondary. - anilkumar gingade On Aug. 2, 2017, 9:12 p.m., Eric Shu wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > 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/ > > > Testing > ------- > > Ongoing precheckin. > > > Thanks, > > Eric Shu > >
