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 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 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 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 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 user kkhatua commented on the issue:
https://github.com/apache/drill/pull/1200
@paul-rogers / @Ben-Zvi , could you review this?
---