Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/1239
One other thing to highlight from an earlier comment. CPU is something that
the user specifies in the DoY config file. That information is passed to YARN
in container requests. This feature asks the user to modify the drill-env.sh
file to enable cgroups to limit CPU. As a result, the user must specify the CPU
limit in two places.
DoY went to extreme lengths to unify memory configuration so it is set in
one place. The assumption was that, since Apache YARN handles cgroups, we've
also got unified CPU specification. But, in this "side-car" approach we don't.
So, would be good to capture the CPU amount from the YARN config as explained
earlier and use that to set the Drill cgroups env vars -- but only if some
"self-enforcing cgroup" flag is enabled in the config (which can be done by
default for the limited MapR YARN.)
---
