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

Ted Yu commented on HDFS-4530:
------------------------------

>From 
>http://javasourcecode.org/html/open-source/jdk/jdk-6u23/sun/misc/VM.java.html :
{code}
    // A user-settable upper limit on the maximum amount of allocatable direct
    // buffer memory.  This value may be changed during VM initialization if
    // "java" is launched with "-XX:MaxDirectMemorySize=<size>".
    //
    // The initial value of this field is arbitrary; during JRE initialization
    // it will be reset to the value specified on the command line, if any,
    // otherwise to Runtime.getRuntime.maxDirectMemory().
    //
    private static long directMemory = 64 * 1024 * 1024;
{code}
                
> return buffer into direct bufferPool in BlockReaderLocal as possible
> --------------------------------------------------------------------
>
>                 Key: HDFS-4530
>                 URL: https://issues.apache.org/jira/browse/HDFS-4530
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: datanode
>    Affects Versions: 3.0.0
>            Reporter: Liang Xie
>            Assignee: Liang Xie
>         Attachments: HDFS-4530.txt, HDFS-4530-v2.txt, HDFS-4530-v3.txt
>
>
> {code}
>   public synchronized void close() throws IOException {
>     dataIn.close();
>     if (checksumIn != null) {
>       checksumIn.close();
>     }
>     if (slowReadBuff != null) {
>       bufferPool.returnBuffer(slowReadBuff);
>       slowReadBuff = null;
>     }
>     if (checksumBuff != null) {
>       bufferPool.returnBuffer(checksumBuff);
>       checksumBuff = null;
>     }
>     startOffset = -1;
>     checksum = null;
>   }
> {code}
> If there's an IOException occurred in dataIn.close(), then the 
> slowReadBuff&checksumBuff could not be returned anymore.  let's make a 
> trivial change to reduce this risk.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to