[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760179#comment-16760179
]
Hudson commented on HDFS-14202:
---
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15879 (See
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760166#comment-16760166
]
Ranith Sardar commented on HDFS-14202:
--
[~elgoiri], please check the recent patch. I have updated
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760170#comment-16760170
]
Ranith Sardar commented on HDFS-14202:
--
Thank you [~elgoiri] :)
>
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760168#comment-16760168
]
Íñigo Goiri commented on HDFS-14202:
Thanks [~RANith] for the patch.
Committed to trunk.
>
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16760167#comment-16760167
]
Íñigo Goiri commented on HDFS-14202:
[^HDFS-14202.005.patch] LGTM.
+1
>
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16759569#comment-16759569
]
Hadoop QA commented on HDFS-14202:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16759534#comment-16759534
]
Ranith Sardar commented on HDFS-14202:
--
[~elgoiri], thanks for reviewing the patch.
I have updated
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16758522#comment-16758522
]
Íñigo Goiri commented on HDFS-14202:
Just try to improve readability for the future:
* Can we replace
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16757989#comment-16757989
]
Ranith Sardar commented on HDFS-14202:
--
{quote}For the assert, can also check for a particular
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16756426#comment-16756426
]
Íñigo Goiri commented on HDFS-14202:
Can we also clarify 21936966?
For the assert, can also check for
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16756360#comment-16756360
]
Ranith Sardar commented on HDFS-14202:
--
[~elgoiri], I have updated the patch. please check once.
>
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16756250#comment-16756250
]
Hadoop QA commented on HDFS-14202:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16754177#comment-16754177
]
Íñigo Goiri commented on HDFS-14202:
For the numbers, I would try to use something more intuitive and
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16752965#comment-16752965
]
Ranith Sardar commented on HDFS-14202:
--
Thanks, [~elgoiri] for the review.
{{{quote}}}
What's the
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16752637#comment-16752637
]
Íñigo Goiri commented on HDFS-14202:
You may want to mark as VisibleForTesting the changes of
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16752115#comment-16752115
]
Hadoop QA commented on HDFS-14202:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16751991#comment-16751991
]
Ranith Sardar commented on HDFS-14202:
--
[~anu] , please review it once.
>
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16751658#comment-16751658
]
Hadoop QA commented on HDFS-14202:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16751466#comment-16751466
]
Ranith Sardar commented on HDFS-14202:
--
[~elgoiri] , thanks for reviewing the patch.
In my current
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742340#comment-16742340
]
Íñigo Goiri commented on HDFS-14202:
The failed unit test seems to be related to
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742259#comment-16742259
]
Hadoop QA commented on HDFS-14202:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16742052#comment-16742052
]
Ranith Sardar commented on HDFS-14202:
--
In DiskBalancer#copyBlocks method, "timeUsed" is calculated
22 matches
Mail list logo