+1, will make people like me that are somewhat afraid of the distributed config pain to look at the code/features.
On 26/02/2016 17:35, Marko Rodriguez wrote: > Hi, > > Bob Briody is doing some work on profile()-step and said: > > "SparkGraphComputerProcessIntegrateTest only takes about 30 seconds to > execute. Why is that integration testing?" > > I thought about it, and I agree. We already have various Spark tests that are > not integration-based. Would people object if I made both: > > SparkGraphComputerProcessIntegrateTest > SparkGraphComputerGroovyProcessIntegrateTest > > ----> > > SparkGraphComputerProcessTest > SparkGraphComputerGroovyProcessTest > > This will add about 1 minute to the "mvn clean install" but will be a lot > safer for people as lots of Serialization/VertexProgram Configuration/etc. > issues get exposed here that you don't see with > TinkerGraphComputerProcessTest. Definitely beneficial to people trying to > submit PRs. > > Thoughts?, > Marko. > > http://markorodriguez.com > >
