[GitHub] drill issue #977: DRILL-5849: Add freemarker lib to dependencyManagement to ...

2017-10-07 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/977 @arina-ielchiieva I refer to java-exec and fmpp modules dependencies resolution. Both AFAIK should not be affected by the change, so I suppose it is another module that has a transitive dependency on

[GitHub] drill issue #977: DRILL-5849: Add freemarker lib to dependencyManagement to ...

2017-10-07 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/977 @vrozov as far as I understand to manage dependency resolution, you can either exclude conflicting dependencies or use dependency management mechanism [1, 2]. I have used dependency

[GitHub] drill pull request #928: DRILL-5716: Queue-driven memory allocation

2017-10-07 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/928#discussion_r143331950 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/DistributedQueryQueue.java --- @@ -165,12 +165,24 @@ public

[GitHub] drill pull request #928: DRILL-5716: Queue-driven memory allocation

2017-10-07 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/928#discussion_r143331929 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/DistributedQueryQueue.java --- @@ -0,0 +1,342 @@ +/* + *