[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread Nanda kumar (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16436171#comment-16436171 ] Nanda kumar commented on HDFS-13413: +1, patch v002 looks good to me. I will commit this shortly. >

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread Bharat Viswanadham (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16435977#comment-16435977 ] Bharat Viswanadham commented on HDFS-13413: --- Thanks for the updated patch. +1 LGTM. >

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16435363#comment-16435363 ] genericqa commented on HDFS-13413: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread Shashikant Banerjee (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16435295#comment-16435295 ] Shashikant Banerjee commented on HDFS-13413: Thanks [~nandakumar131] for the review . patch v2

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread Nanda kumar (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16435240#comment-16435240 ] Nanda kumar commented on HDFS-13413: Thanks [~shashikant] for working on this. Just a small

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16435047#comment-16435047 ] genericqa commented on HDFS-13413: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-12 Thread Shashikant Banerjee (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434997#comment-16434997 ] Shashikant Banerjee commented on HDFS-13413: Thanks [~bharatviswa], for the review. Patch v1

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-11 Thread Bharat Viswanadham (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434947#comment-16434947 ] Bharat Viswanadham commented on HDFS-13413: --- Hi [~shashikant]  Thanks for reporting and working

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-11 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434798#comment-16434798 ] genericqa commented on HDFS-13413: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-13413) ClusterId and DatanodeUuid should be marked mandatory fileds in SCMRegisteredCmdResponseProto

2018-04-11 Thread Shashikant Banerjee (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-13413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434697#comment-16434697 ] Shashikant Banerjee commented on HDFS-13413: Patch v0 makes DatanodeUUid and Cluster Id in the