Re: DistributionArchUnitTest OutOfMemoryError

2019-11-22 Thread Kirk Lund
LoggingDependenciesTest is also an archunit test in geode-core. I'm going to move LoggingDependenciesTest to integrationTest -- testing the direction of dependency between two geode modules seems to fit the definition of "integration". On Thu, Nov 21, 2019 at 12:09 PM Dan Smith wrote: > We've di

Re: DistributionArchUnitTest OutOfMemoryError

2019-11-21 Thread Dan Smith
We've disabled this test for now, so you shouldn't get an OOME with the latest develop. -Dan On Wed, Nov 20, 2019, 4:55 PM Michael Oleske wrote: > I've hit the same error as Kirk. My lazy solution was to just rerun > `./gradlew build` cause that would pass on the second time. > > -michael > >

Re: DistributionArchUnitTest OutOfMemoryError

2019-11-20 Thread Michael Oleske
I've hit the same error as Kirk. My lazy solution was to just rerun `./gradlew build` cause that would pass on the second time. -michael On Wed, Nov 20, 2019 at 4:53 PM Dan Smith wrote: > We recently added this test. It passes for me, but I will look into it. It > is scanning classes, so that

Re: DistributionArchUnitTest OutOfMemoryError

2019-11-20 Thread Dan Smith
We recently added this test. It passes for me, but I will look into it. It is scanning classes, so that may be your oome. On Wed, Nov 20, 2019, 4:00 PM Kirk Lund wrote: > Any ideas why DistributionArchUnitTest would run OutOfMemoryError when > doing a "./gradlew build"? > > I think we should mov

DistributionArchUnitTest OutOfMemoryError

2019-11-20 Thread Kirk Lund
Any ideas why DistributionArchUnitTest would run OutOfMemoryError when doing a "./gradlew build"? I think we should move any unit tests that run OutOfMemoryError out of unit tests (to integration tests maybe?). > Task :geode:geode-core:test Heap dump file created [957877145 bytes in 17.227 secs]