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

Rushabh S Shah commented on HDFS-12143:
---------------------------------------

{code:title= INodeWithAdditionalFields.java|borderStyle=solid}
protected void removeFeature(Feature f)
{
  ...
   if (overflow || j != size - 1) {
      throwFeatureNotFoundException(f);
    }
  ...
}
{code}
 j is not equal to {{size-1}}, only if we found multiple features since we 
break out when j == size-1.

> Improve performance of getting and removing inode features
> ----------------------------------------------------------
>
>                 Key: HDFS-12143
>                 URL: https://issues.apache.org/jira/browse/HDFS-12143
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode
>    Affects Versions: 2.8.0
>            Reporter: Daryn Sharp
>            Assignee: Daryn Sharp
>         Attachments: HDFS-12143.patch
>
>
> Getting a feature uses an iterator which is less performant than an indexed 
> for loop.  Feature lookups are becoming more prolific so cycles count.
> Removing a feature requires building a string for up to 3 precondition 
> checks.  The worst case of 3 is the penalty for a successful removal.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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

Reply via email to