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

Wei-Chiu Chuang commented on HDFS-12245:
----------------------------------------

[~adam.antal] -- if you do the math, 2^14-1 is 16383 :)
I think let's keep it as is, you can add a comment after that saying it's 
16384. Like the following:
{code}
public static final long LAST_RESERVED_ID = 2 << 14 - 1; // 16383
public static final long ROOT_INODE_ID = LAST_RESERVED_ID + 1; // 16384
{code}

> Update INodeId javadoc
> ----------------------
>
>                 Key: HDFS-12245
>                 URL: https://issues.apache.org/jira/browse/HDFS-12245
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: namenode
>            Reporter: Wei-Chiu Chuang
>            Assignee: Adam Antal
>            Priority: Major
>         Attachments: HDFS-12245.001.patch
>
>
> The INodeId javadoc states that id 1 to 1000 is reserved and root inode id 
> start from 1001. That is no longer true after HDFS-4434.
> Also, it's a little weird in INodeId
> {code}
>   public static final long LAST_RESERVED_ID = 2 << 14 - 1;
>   public static final long ROOT_INODE_ID = LAST_RESERVED_ID + 1;
> {code}
> It seems the intent was for LAST_RESERVED_ID to be (2^14) - 1 = 32767. But 
> due to Java operator precedence, LAST_RESERVED_ID = 2^(14-1) = 16384. Maybe 
> it doesn't matter, not sure.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to