Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-03 Thread anilkumar gingade
--- 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/a

Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-02 Thread Eric Shu
> On Aug. 1, 2017, 11:12 p.m., anilkumar gingade wrote: > > geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRTransactionDUnitTest.java > > Lines 1164 (patched) > > > > > > Since only one test using

Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-02 Thread Eric Shu
--- 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,

Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-02 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61281/#review182006 --- Ship it! Ship It! - Darrel Schneider On Aug. 1, 2017, 2:41 p

Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-01 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61281/#review181924 --- geode-core/src/test/java/org/apache/geode/internal/cache/execute/

Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-01 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61281/ --- Review request for geode, anilkumar gingade, Darrel Schneider, Lynn Gallinat, an