[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16489426#comment-16489426
]
Hudson commented on HDFS-13611:
---
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14283 (See
[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16488397#comment-16488397
]
Xiao Chen commented on HDFS-13611:
--
Good catch and thanks for the review Todd!
> Unsafe use of Text as a
[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487967#comment-16487967
]
genericqa commented on HDFS-13611:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487867#comment-16487867
]
Todd Lipcon commented on HDFS-13611:
+1, looks good. While we're at it, we could reduce one extra
[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487864#comment-16487864
]
Andrew Wang commented on HDFS-13611:
Even simpler patch attached, if we make a copy of the Text for
[
https://issues.apache.org/jira/browse/HDFS-13611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487849#comment-16487849
]
Andrew Wang commented on HDFS-13611:
Simple patch attached, getFixedByteString(Text) wasn't even