drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-1942) Improve off-heap memory usage tracking
Date Thu, 10 Sep 2015 21:14:45 GMT

    [ https://issues.apache.org/jira/browse/DRILL-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14739602#comment-14739602
] 

ASF GitHub Bot commented on DRILL-1942:
---------------------------------------

Github user jinfengni commented on a diff in the pull request:

    https://github.com/apache/drill/pull/120#discussion_r39214700
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/vector/BitVector.java ---
    @@ -211,8 +220,11 @@ public TransferPair makeTransferPair(ValueVector to) {
     
       public void transferTo(BitVector target) {
         target.clear();
    +    if (target.data != null) {
    +      target.data.release();
    --- End diff --
    
    I think the current form is good. I'll merge the patch at it is. 


> Improve off-heap memory usage tracking
> --------------------------------------
>
>                 Key: DRILL-1942
>                 URL: https://issues.apache.org/jira/browse/DRILL-1942
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Execution - Relational Operators
>            Reporter: Chris Westin
>            Assignee: Chris Westin
>             Fix For: 1.2.0
>
>         Attachments: DRILL-1942.1.patch.txt, DRILL-1942.2.patch.txt, DRILL-1942.3.patch.txt
>
>
> We're using a lot more memory than we think we should. We may be leaking it, or not releasing
it as soon as we could. 
> This is a call to come up with some improved tracking so that we can get statistics out
about exactly where we're using it, and whether or not we can release it earlier.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message