Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142573753
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
@@ -351,7 +351,35 @@
MAX_QUERY_MEMORY_PER_NODE_KEY, 1024 * 1024, Long.MAX_VALUE);
/**
- * Minimum memory alocated to each buffered operator instance.
+ * Alternative way to compute per-query-per-node memory as a percent
+ * of the total available system memory.
+ * <p>
+ * Suggestion for computation.
+ * <ul>
+ * <li>Assume an allowance for non-managed operators. Default assumption:
+ * 50%</li>
+ * <li>Assume a desired number of concurrent queries. Default assumption:
+ * 10.</li>
+ * <li>The value of this parameter is<br>
+ * (1 - non-managed allowance) / concurrency</li>
--- End diff --
Would it be simpler if working in a 100 base for the percent; i.e. "1" here
be "100", to work in percentage numbers as integers (0...100) instead of a
decimal fraction ?
---
