[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-12 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1200 @Ben-Zvi .. batch committers would be doing that. I've got the feed back that it helps to preserve the commit-&-comment history and the squashing can be done in pre-commit merge branch.

[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-10 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1200 @Ben-Zvi / @paul-rogers I've updated the PR with changes based on your comments. Waiting for a review. ---

[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-04 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1200 @Ben-Zvi / @paul-rogers I discovered that the path `/cgroup` is unique only to my installation of the package on CentOS. The standard path is `/sys/fs/cgroup`. So, I've made changes that

[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-04 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1200 @paul-rogers very good point about the issue of the heartbeat thread competing with other Drillbit process threads for CPU. When writing up the documentation for this, we'll have to ensure that

[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-03 Thread Ben-Zvi
Github user Ben-Zvi commented on the issue: https://github.com/apache/drill/pull/1200 Could there be users with an older Linux (pre 4.5, circa March 2016) which does not support cgroups V2 ? ---

[GitHub] drill issue #1200: DRILL-143: Support CGROUPs resource management

2018-04-03 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1200 @paul-rogers / @Ben-Zvi , could you review this? ---