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

Ayush Saxena commented on HDFS-12733:
-------------------------------------

Thanx [~hexiaoqiao]
can you check this :

{code:java}
else if (dirNames.isEmpty()) {
      dirNames = Collections.singletonList(
          DFSConfigKeys.DFS_NAMENODE_EDITS_DIR_DEFAULT);
    }
{code}
This block could have pitched in if noShared was empty, Now this will not 
happen, Is this desirable? 

{code:java}
    if (!noSharedEditDirs.isEmpty()) {
{code}

Shouldn't we check have a check that editsDirs isn't empty as well?

[~shv] do give a check, if you have time if your previous concerns are solved 
with this approach.

> Option to disable to namenode local edits
> -----------------------------------------
>
>                 Key: HDFS-12733
>                 URL: https://issues.apache.org/jira/browse/HDFS-12733
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode, performance
>            Reporter: Brahma Reddy Battula
>            Assignee: Xiaoqiao He
>            Priority: Major
>         Attachments: HDFS-12733-001.patch, HDFS-12733-002.patch, 
> HDFS-12733-003.patch, HDFS-12733.004.patch, HDFS-12733.005.patch, 
> HDFS-12733.006.patch, HDFS-12733.007.patch, HDFS-12733.008.patch, 
> HDFS-12733.009.patch
>
>
> As of now, Edits will be written in local and shared locations which will be 
> redundant and local edits never used in HA setup.
> Disabling local edits gives little performance improvement.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to