hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hari Mankude (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-3213) JournalDaemon (server) should persist the cluster id and nsid in the storage directory
Date Wed, 11 Apr 2012 20:25:17 GMT

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

Hari Mankude commented on HDFS-3213:

bq. JournalService.fence(..), should the setupStorage(..) call be after verifyFence(..) and
One of the advantages of doing this earlier is that verify() will check for results automatically.

bq.   JournalService.conf is only used in setupStorage(..). Instead of adding it as a field,
it is better to replace it with NNStorage since the conf values are fixed. We don't have to
create the objects in setupStorage(..).
Let us leave this as is for now. This will require rework once edits dir format for journals
is decided.
bq.    Need to add Assert.fail() after fence(..) in the new tests. Otherwise, the test will
pass even if fence(..) does not throw exceptions. I think it is better to print out the exception
in the catch-blocks instead of leaving them empty.

Done for both, assert.fail() and printfs.

bq. In addition, we need to think about how to synchronize the methods in JournalService.
Do you want to do it here or in a separated issue?
Again, I would like to defer this for later. I will file a jira for this.
> JournalDaemon (server) should persist the cluster id and nsid in the storage directory
> --------------------------------------------------------------------------------------
>                 Key: HDFS-3213
>                 URL: https://issues.apache.org/jira/browse/HDFS-3213
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha, name-node
>            Reporter: Hari Mankude
>            Assignee: Hari Mankude
>         Attachments: hdfs-3213-1.patch, hdfs-3213-2.patch, hdfs-3213.patch

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


View raw message