[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758761#comment-16758761
]
Hadoop QA commented on HDFS-14250:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758715#comment-16758715
]
Erik Krogen commented on HDFS-14250:
Great ! +1 from me pending Jenkins, will commit
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758707#comment-16758707
]
Chao Sun commented on HDFS-14250:
-
Thanks [~xkrogen]! Attached patch v1 to address the co
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758643#comment-16758643
]
Erik Krogen commented on HDFS-14250:
v000 patch LGTM, only minor nit is that I would
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16757742#comment-16757742
]
Chao Sun commented on HDFS-14250:
-
Yes, I think we can just use {{activeOnly}} for this t
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16757722#comment-16757722
]
Erik Krogen commented on HDFS-14250:
I don't think we want to make it write anything
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16757712#comment-16757712
]
Chao Sun commented on HDFS-14250:
-
[~xkrogen] you are right. Maybe {{msync}} can simply b
[
https://issues.apache.org/jira/browse/HDFS-14250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16757704#comment-16757704
]
Erik Krogen commented on HDFS-14250:
I think #2 is not necessary -- this will be hand
