Re: Removal of distributed transaction tests

2017-02-09 Thread Kirk Lund
If we think we'll eventually finish the feature then that sounds good. On Thu, Feb 9, 2017 at 2:01 PM, Swapnil Bawaskar wrote: > Implementation for distributed transactions was started but has not been > completed yet. The tests may only be testing what has been implemented so > far, so I would

Re: Removal of distributed transaction tests

2017-02-09 Thread Swapnil Bawaskar
Implementation for distributed transactions was started but has not been completed yet. The tests may only be testing what has been implemented so far, so I would vote not to remove these tests. On Thu, Feb 9, 2017 at 1:57 PM Kirk Lund wrote: > My reasoning is simple: Apache Geode does not curren

Re: Removal of distributed transaction tests

2017-02-09 Thread Kirk Lund
My reasoning is simple: Apache Geode does not currently have a distributed transactions feature. On Thu, Feb 9, 2017 at 12:03 PM, Dan Smith wrote: > What's wrong with these tests? Are you saying there is coverage elsewhere? > > -Dan > > On Thu, Feb 9, 2017 at 12:02 PM, Kirk Lund wrote: > > > D

Re: Removal of distributed transaction tests

2017-02-09 Thread Dan Smith
What's wrong with these tests? Are you saying there is coverage elsewhere? -Dan On Thu, Feb 9, 2017 at 12:02 PM, Kirk Lund wrote: > Do we really want our nightly build and precheckin runs to waste time > running these tests? > > DistributedTransactionDUnitTest > DistTXDebugDUnitTest > DistTXOrd