[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-08 Thread Andrew Wang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14125240#comment-14125240 ] Andrew Wang commented on HDFS-6951: --- Overall looks good to me, just some test nits:

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-08 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14125639#comment-14125639 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-08 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14126034#comment-14126034 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-08 Thread Charles Lamb (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14126177#comment-14126177 ] Charles Lamb commented on HDFS-6951:

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-07 Thread Andrew Wang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14125230#comment-14125230 ] Andrew Wang commented on HDFS-6951: --- Blech, it looks like test-patch doesn't like binary

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14124443#comment-14124443 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-05 Thread Andrew Wang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14123703#comment-14123703 ] Andrew Wang commented on HDFS-6951: --- Charles, do you mind rebasing this? It doesn't apply

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-05 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14124275#comment-14124275 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-02 Thread Yi Liu (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14118149#comment-14118149 ] Yi Liu commented on HDFS-6951: -- Thanks [~clamb], looks good to me, just one comment: yes,

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14118488#comment-14118488 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14118936#comment-14118936 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-02 Thread Charles Lamb (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14119015#comment-14119015 ] Charles Lamb commented on HDFS-6951:

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-09-02 Thread Yi Liu (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14119097#comment-14119097 ] Yi Liu commented on HDFS-6951: -- Thanks [~clamb], It's OK for me. +1 (non-binding). Saving

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-29 Thread Charles Lamb (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14115204#comment-14115204 ] Charles Lamb commented on HDFS-6951: I ran these two tests on my local machine:

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14114243#comment-14114243 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14114722#comment-14114722 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14114763#comment-14114763 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14114765#comment-14114765 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14112681#comment-14112681 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-27 Thread Andrew Wang (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14112947#comment-14112947 ] Andrew Wang commented on HDFS-6951: --- Hi Charles, thanks for taking a look at this. Had

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14112954#comment-14112954 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-27 Thread Yi Liu (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14113193#comment-14113193 ] Yi Liu commented on HDFS-6951: -- Hi [~clamb], thanks for the fix :) In original design, we

[jira] [Commented] (HDFS-6951) Saving namespace and restarting NameNode will remove existing encryption zones

2014-08-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HDFS-6951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14113313#comment-14113313 ] Hadoop QA commented on HDFS-6951: - {color:red}-1 overall{color}. Here are the results of