diff options
| author | Paul Rogers <progers@maprtech.com> | 2017-04-06 13:57:19 -0700 |
|---|---|---|
| committer | Paul Rogers <progers@maprtech.com> | 2017-06-21 10:02:16 -0700 |
| commit | 90f43bff7a01eaaee6c8861137759b05367dfcf3 (patch) | |
| tree | 9e2e0b0fdfa2514d4ab5d48272b68ac8a52be1eb /exec/java-exec/pom.xml | |
| parent | c16e5f8072f3e5d18157767143f9ccc7669c4380 (diff) | |
DRILL-5325: Unit tests for the managed sort
Uses the sub-operator test framework (DRILL-5318), including the test
row set abstraction (DRILL-5323) to enable unit testing of the
“managed” external sort. This PR allows early review of the code, but
cannot be pulled until the dependencies (mentioned above) are pulled.
Refactors the external sort code into small chunks that can be unit
tested, then “wraps” that code in tests for all interesting data types,
record batch sizes, and so on.
Refactors some of the operator definitions to more easily allow
programmatic setup in the unit tests.
Fixes a number of bugs discovered by the unit tests. The biggest
changes were in the new code: the code that computes spilling and
merging based on memory levels.
Otherwise, although GitHub will show many files change, most of the
changes are simply moving blocks of code around to create smaller units
that can be tested independently.
Includes a refactoring of the code that does spilling, along with a
complete set of low-level unit tests.
Excludes long-running sort tests.
Defines a test category for long-running tests.
First attempt to provide a way to run such tests from Maven.
closes #808
Diffstat (limited to 'exec/java-exec/pom.xml')
| -rw-r--r-- | exec/java-exec/pom.xml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/exec/java-exec/pom.xml b/exec/java-exec/pom.xml index 3b1ac4c25..cd287aa58 100644 --- a/exec/java-exec/pom.xml +++ b/exec/java-exec/pom.xml @@ -532,6 +532,7 @@ <includes> <include>**/TestLargeFileCompilation.java</include> </includes> + <groups>org.apache.drill.test.SecondaryTest</groups> </configuration> </execution> </executions> @@ -732,9 +733,13 @@ <phase>test</phase> <goals><goal>test</goal></goals> <configuration> + <!-- Legacy: exclude by file name --> <excludes> <exclude>**/TestLargeFileCompilation.java</exclude> </excludes> + <!-- Modern: include using the @Category annotation. + See the Javadoc for SecondaryTest for details. --> + <excludedGroups>org.apache.drill.test.SecondaryTest</excludedGroups> </configuration> </execution> </executions> |