[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-13 Thread LiXin Ge (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614401#comment-16614401 ] LiXin Ge commented on HDDS-369: --- Hi [~elek], I have created HDDS-449 to add the null check as we discussed,

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-13 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16613310#comment-16613310 ] Elek, Marton commented on HDDS-369: --- [~GeLiXin] You are right. For the unit tests we need to ensure that

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-12 Thread LiXin Ge (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16612943#comment-16612943 ] LiXin Ge commented on HDDS-369: --- Thanks for the commets [~elek]. Actually I find this doubts when I add unit

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-12 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16611796#comment-16611796 ] Elek, Marton commented on HDDS-369: --- Thanks the commit [~hanishakoneru] and the good question [~GeLiXin].

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-08 Thread LiXin Ge (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607934#comment-16607934 ] LiXin Ge commented on HDDS-369: --- Hi [~elek], sorry for the late review, is there have chance to introduce a

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-04 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603708#comment-16603708 ] Hudson commented on HDDS-369: - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14875 (See

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-04 Thread Hanisha Koneru (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603653#comment-16603653 ] Hanisha Koneru commented on HDDS-369: - Committed to trunk. Thanks [~elek] for working on this and

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-04 Thread Hanisha Koneru (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603535#comment-16603535 ] Hanisha Koneru commented on HDDS-369: - Thanks for updating the patch [~elek]. Patch v06 LGTM. I have

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-03 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602136#comment-16602136 ] Hadoop QA commented on HDDS-369: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-03 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602010#comment-16602010 ] Hadoop QA commented on HDDS-369: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-09-03 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16601951#comment-16601951 ] Elek, Marton commented on HDDS-369: --- Yes, for some reason v005 has not been picked by jenkins. The last

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-31 Thread Ajay Kumar (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16599073#comment-16599073 ] Ajay Kumar commented on HDDS-369: - test failure seems related. > Remove the containers of a dead node from

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

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

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

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

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-30 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16597282#comment-16597282 ] Elek, Marton commented on HDDS-369: --- Patch is rebased to the trunk. > Remove the containers of a dead

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

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

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-28 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16595295#comment-16595295 ] Elek, Marton commented on HDDS-369: --- Thanks the review [~nandakumar131] and [~ajayydv]. I uploaded a new

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-27 Thread Ajay Kumar (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16594120#comment-16594120 ] Ajay Kumar commented on HDDS-369: - [~elek] Patch LGTM. Minor nit: Change log statement to parametrized

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-27 Thread Nanda kumar (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16594082#comment-16594082 ] Nanda kumar commented on HDDS-369: -- [~elek], Thanks for working on this. Instead of using

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-24 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591744#comment-16591744 ] genericqa commented on HDDS-369: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-24 Thread Elek, Marton (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591655#comment-16591655 ] Elek, Marton commented on HDDS-369: --- Thanks [~hanishakoneru] to check it. The shiny new patch is just

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

2018-08-23 Thread Hanisha Koneru (JIRA)
[ https://issues.apache.org/jira/browse/HDDS-369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16590694#comment-16590694 ] Hanisha Koneru commented on HDDS-369: - Thanks for working on this [~elek]. The patch LGTM overall. Can

[jira] [Commented] (HDDS-369) Remove the containers of a dead node from the container state map

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