[
https://issues.apache.org/jira/browse/HDFS-13300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413946#comment-16413946
]
Nanda kumar commented on HDFS-13300:
------------------------------------
In patch v004 we start ObjectStoreRestPlugin before HdslDatanodeService so that
ObjectStoreRestPlugin will update the correct port in DatanodeDetails before
it's sent to SCM.
Flow:
ObjectStoreRestPlugin is configured first, followed by HdslDatanodeService
# ObjectStoreRestPlugin#constructor
## Checks if Ozone is enabled
# HdslDatanodeService#constructor
## Checks if Ozone is enabled
## Constructs DatanodeDetails instance
# ObjectStoreRestPlugin#start
## Creates ObjectStoreHandler
## Create and starts ObjectStoreRestHttpServer
## Update REST port in DatanodeDetails
# HdslDatanodeService#start
## Create and starts DatanodeStateMachine
<registration of datanode with SCM is done later>
{quote}but I got NPE when I tested patch v3
{quote}
This is because Ratis needs IP for peerID creation, this is also fixed in patch
v004.
> Ozone: Remove DatanodeID dependency from HDSL and Ozone
> --------------------------------------------------------
>
> Key: HDFS-13300
> URL: https://issues.apache.org/jira/browse/HDFS-13300
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ozone
> Reporter: Nanda kumar
> Assignee: Nanda kumar
> Priority: Major
> Attachments: HDFS-13300-HDFS-7240.000.patch,
> HDFS-13300-HDFS-7240.001.patch, HDFS-13300-HDFS-7240.002.patch,
> HDFS-13300-HDFS-7240.003.patch, HDFS-13300-HDFS-7240.004.patch
>
>
> DatanodeID has been modified to add HDSL/Ozone related information
> previously. This jira is to remove DatanodeID dependency from HDSL/Ozone to
> make it truly pluggable without having the need to modify DatanodeID.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
