[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-13 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16863603#comment-16863603 ] Hudson commented on HDFS-14560: --- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #16743 (See

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-13 Thread JIRA
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16863594#comment-16863594 ] Íñigo Goiri commented on HDFS-14560: Thanks [~sodonnell] for the patch and [~hexiaoqiao] for the

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862593#comment-16862593 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862493#comment-16862493 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862486#comment-16862486 ] Stephen O'Donnell commented on HDFS-14560: -- Deleted it and re-uploaded with the new name. Many

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread JIRA
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862483#comment-16862483 ] Íñigo Goiri commented on HDFS-14560: [^HDFS-14560.005.patch] LGTM. +1 Do you mind labeling the patch

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862478#comment-16862478 ] Stephen O'Donnell commented on HDFS-14560: -- When committing this, it would be great if we could

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862385#comment-16862385 ] Stephen O'Donnell commented on HDFS-14560: -- I just pushed up v005 with the change to static for

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread He Xiaoqiao (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862326#comment-16862326 ] He Xiaoqiao commented on HDFS-14560: +1 for [^HDFS-14560.004.patch]. Thanks [~sodonnell] for your

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread JIRA
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862296#comment-16862296 ] Íñigo Goiri commented on HDFS-14560: Just a minor comment, let's make {{ensurePositiveInt()}} static.

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16862003#comment-16862003 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-12 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861857#comment-16861857 ] Stephen O'Donnell commented on HDFS-14560: -- I have tightened up the code around the

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861588#comment-16861588 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread JIRA
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861577#comment-16861577 ] Íñigo Goiri commented on HDFS-14560: Thanks [~sodonnell] for tackling the comments. {quote} OK, I had

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861447#comment-16861447 ] Stephen O'Donnell commented on HDFS-14560: -- [~hexiaoqiao] I agree that it would nice for as many

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861362#comment-16861362 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread He Xiaoqiao (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861224#comment-16861224 ] He Xiaoqiao commented on HDFS-14560: another thought based on this issue: there are many parameters

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread JIRA
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861206#comment-16861206 ] Íñigo Goiri commented on HDFS-14560: Thanks [~sodonnell] for the patch. A few minor comments: * I

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread Stephen O'Donnell (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861186#comment-16861186 ] Stephen O'Donnell commented on HDFS-14560: -- I don't think the test failures are related. I have

[jira] [Commented] (HDFS-14560) Allow block replication parameters to be refreshable

2019-06-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-14560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16861122#comment-16861122 ] Hadoop QA commented on HDFS-14560: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||