You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by ilooner <gi...@git.apache.org> on 2018/03/19 21:11:46 UTC

[GitHub] drill pull request #1176: DRILL-6275: Fixed direct memory reporting in sys.m...

GitHub user ilooner opened a pull request:

    https://github.com/apache/drill/pull/1176

    DRILL-6275: Fixed direct memory reporting in sys.memory.

    @kkhatua Thanks for pinpointing the root cause! Please review. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ilooner/drill DRILL-6275

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/1176.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1176
    
----
commit 7f65b7d4b4b9e42dc3597ac9758c39c6ce0903b7
Author: Timothy Farkas <ti...@...>
Date:   2018-03-19T20:16:37Z

    DRILL-6275: Fixed direct memory reporting in sys.memory.

----


---

[GitHub] drill issue #1176: DRILL-6275: Fixed direct memory reporting in sys.memory.

Posted by ilooner <gi...@git.apache.org>.

Github user ilooner commented on the issue:

    https://github.com/apache/drill/pull/1176
  
    Tested fix manually on my laptop.


---

[GitHub] drill pull request #1176: DRILL-6275: Fixed direct memory reporting in sys.m...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/1176


---

[GitHub] drill issue #1176: DRILL-6275: Fixed direct memory reporting in sys.memory.

Posted by kkhatua <gi...@git.apache.org>.

Github user kkhatua commented on the issue:

    https://github.com/apache/drill/pull/1176
  
    LGTM. Verified on a 4-node setup with running queries.
    +1


---