[
https://issues.apache.org/jira/browse/HDFS-3202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13246936#comment-13246936
]
Suresh Srinivas commented on HDFS-3202:
---------------------------------------
+1 for the patch. I think this is trivial enough and unit test is an overkill.
One quick comment - do we need second variant of the NamespaceInfo constructor?
If it is not used at many places, can we just have one constructor?
> NamespaceInfo PB translation drops build version
> ------------------------------------------------
>
> Key: HDFS-3202
> URL: https://issues.apache.org/jira/browse/HDFS-3202
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: data-node
> Affects Versions: 2.0.0
> Reporter: Aaron T. Myers
> Assignee: Aaron T. Myers
> Attachments: HDFS-3202.patch
>
>
> The PBHelper#convert(NamespaceInfoProto) function doesn't pass the build
> version from the NamespaceInfoProto to the created NamespaceInfo object.
> Instead, the NamespaceInfo constructor gets the build version using the
> static function Storage#getBuildVersion. DNs also use this static function to
> determine their own build version. This means that the check the DN does to
> compare its own build version to that of the NN always passes, regardless of
> what build version exists on the NN.
--
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
