Repository: drill Updated Branches: refs/heads/master 23042dec6 -> 894c0f58e
DILL-6148: TestSortSpillWithException is sometimes failing. closes #1120 Project: http://git-wip-us.apache.org/repos/asf/drill/repo Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/5d4fbd1e Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/5d4fbd1e Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/5d4fbd1e Branch: refs/heads/master Commit: 5d4fbd1e65ec197780088889eec81e01d29f1167 Parents: 23042de Author: Hanumath Rao Maduri <hanu....@gmail.com> Authored: Fri Feb 9 14:13:04 2018 -0800 Committer: Vitalii Diravka <vitalii.dira...@gmail.com> Committed: Fri Feb 16 17:24:04 2018 +0000 ---------------------------------------------------------------------- .../exec/physical/impl/xsort/TestSortSpillWithException.java | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/drill/blob/5d4fbd1e/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortSpillWithException.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortSpillWithException.java b/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortSpillWithException.java index cd3f5fd..5ab29de 100644 --- a/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortSpillWithException.java +++ b/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortSpillWithException.java @@ -59,6 +59,11 @@ public class TestSortSpillWithException extends ClusterTest { ClusterFixtureBuilder builder = ClusterFixture.builder(dirTestWatcher) .configProperty(ExecConstants.EXTERNAL_SORT_SPILL_THRESHOLD, 1) // Unmanaged .configProperty(ExecConstants.EXTERNAL_SORT_SPILL_GROUP_SIZE, 1) // Unmanaged + // Using EXTERNAL_SORT_MAX_MEMORY to set low values of memory for SORT instead + // of lowering MAX_QUERY_MEMORY_PER_NODE_KEY because computation of operator memory + // cannot go lower than MIN_MEMORY_PER_BUFFERED_OP (the default value of this parameter + // is 40MB). The 40MB memory is sufficient for this testcase to run sort without spilling. + .configProperty(ExecConstants.EXTERNAL_SORT_MAX_MEMORY, 10 * 1024 * 1024) .sessionOption(ExecConstants.MAX_QUERY_MEMORY_PER_NODE_KEY, 60 * 1024 * 1024) // Spill early // Prevent the percent-based memory rule from second-guessing the above. .sessionOption(ExecConstants.PERCENT_MEMORY_PER_QUERY_KEY, 0.0)