Hi Vladimir, Thanks for working on this.
I have a quick question regarding fast/slow tests. Currently we have two ways of running/skipping a slow test: * using the SlowTests annotation and using JUnit categories; * using the system property calcite.test.slow. I was planning (WIP in [1]) to unify the two by annotating all slow tests with the appropriate annotation. Does this make sense considering the ongoing work of moving to Gradle and JUnit5? Are you planning to keep/use these kind of annotations? Best, Stamatis [1] https://github.com/apache/calcite/pull/1375 On Mon, Nov 11, 2019 at 11:30 PM Vladimir Sitnikov < [email protected]> wrote: > As you might know, Avatica is already migrated to Gradle. > > On top of that, Calcite is almost there as well: > https://github.com/apache/calcite/pull/1571 > > The missing bit is probably the notion of tests "fast/slow/etc/etc". > Unfortunately, there are tests that assert on folder layout like > "target/test-classes". > I'm going to ignore those tests for now, and fix them as pom.xml files are > removed. > > I'm going to configure Gradle to use JUnit5 engine to run the tests. > > In the meantime, everybody's welcome to preview Gradle build. > > Vladimir >
