hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yiqun Lin (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-12739) Add Support for SCM --init command
Date Wed, 01 Nov 2017 06:23:02 GMT

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

Yiqun Lin commented on HDFS-12739:

Thanks for working on this, [~shashikant]. The following are some comments from me:

# The usage of {{GENCLUSTERID}} is missing in {{USAGE}}.
# The return value of {{StorageContainerManager#scmInit}} looks confused. Scm initial successful,
the method return false, if failed, return true. Can we make a change on this? And let {{aborted
= !scmInit(conf);}}.
# Following line only prints cluster id and no other description, would you make a change?
      private static boolean scmInit(OzoneConfiguration conf) throws IOException {
        if (state != StorageState.NORMAL) {
          try {
            clusterId = StartupOption.INIT.getClusterId();
            if (clusterId == null || clusterId.isEmpty()) {
              //Generate a new cluster id
              clusterId = SCMStorage.newClusterID();
            System.out.println(clusterId);   <=====
            return false;
# Here we introduces new scm commands, we need to add some test to verify the behaviour of
these commands.

> Add Support for SCM --init command
> ----------------------------------
>                 Key: HDFS-12739
>                 URL: https://issues.apache.org/jira/browse/HDFS-12739
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: HDFS-7240
>    Affects Versions: HDFS-7240
>            Reporter: Shashikant Banerjee
>            Assignee: Shashikant Banerjee
>            Priority: Major
>         Attachments: HDFS-12739-HDFS-7240.001.patch, HDFS-12739-HDFS-7240.002.patch,
> SCM --init command will generate cluster ID and persist it locally. The same cluster
Id will be shared with KSM and the datanodes. IF the cluster Id is already available in the
locally available version file, it will just read the cluster Id .

This message was sent by Atlassian JIRA

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

View raw message