Yiqun Lin created HDFS-13598:
--------------------------------

             Summary: Reduce unnecessary byte-to-string transform operation in 
INodesInPath#toString
                 Key: HDFS-13598
                 URL: https://issues.apache.org/jira/browse/HDFS-13598
             Project: Hadoop HDFS
          Issue Type: Improvement
    Affects Versions: 3.1.0
            Reporter: Yiqun Lin


Every time we invoke {{INodesInPath#toString()}}, we will trigger one time 
byte-to-string transform operation.
{code:java}
private String toString(boolean vaildateObject) {
    if (vaildateObject) {
      validate();
    }

    final StringBuilder b = new StringBuilder(getClass().getSimpleName())
        .append(": path = ").append(DFSUtil.byteArray2PathString(path))
        .append("\n  inodes = ");
        ...
}
{code}
But actually we only need to do at most one time. In {{INodesInPath}}, it has 
defined one String type variable named {{pathname}} for storing the path string 
value. Here we can use {{getPath()}} to to replace 
{{DFSUtil.byteArray2PathString(path)}}.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to