[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173770#comment-14173770
]
Hudson commented on YARN-2312:
--
FAILURE: Integrated in Hadoop-Mapreduce-trunk #1928 (See
[htt
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173760#comment-14173760
]
Hudson commented on YARN-2312:
--
FAILURE: Integrated in Hadoop-Hdfs-trunk #1903 (See
[https://
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173640#comment-14173640
]
Hudson commented on YARN-2312:
--
FAILURE: Integrated in Hadoop-Yarn-trunk #713 (See
[https://b
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173152#comment-14173152
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173076#comment-14173076
]
Hudson commented on YARN-2312:
--
FAILURE: Integrated in Hadoop-trunk-Commit #6267 (See
[https:
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14173056#comment-14173056
]
Jian He commented on YARN-2312:
---
+1, committing
> Marking ContainerId#getId as deprecated
>
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171470#comment-14171470
]
Jason Lowe commented on YARN-2312:
--
Sorry for the late reply. +1 lgtm as well. I noticed
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14167243#comment-14167243
]
Jian He commented on YARN-2312:
---
the latest patch looks good to me, the warnings should not b
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14166641#comment-14166641
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14165530#comment-14165530
]
Jian He commented on YARN-2312:
---
took look at the patch again, seems we should do parseLong f
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163248#comment-14163248
]
Tsuyoshi OZAWA commented on YARN-2312:
--
s/YARN-6115/MAPREDUCE-6115/
> Marking Contain
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163120#comment-14163120
]
Tsuyoshi OZAWA commented on YARN-2312:
--
The test failure is not related - the failure
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14161902#comment-14161902
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160653#comment-14160653
]
Jian He commented on YARN-2312:
---
Patch looks good to me too, thanks Jason for reviewing the p
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160481#comment-14160481
]
Jason Lowe commented on YARN-2312:
--
Patch looks good overall. Just two minor nits:
The c
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159425#comment-14159425
]
Tsuyoshi OZAWA commented on YARN-2312:
--
The test failure of TestPipeApplication is not
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159241#comment-14159241
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159046#comment-14159046
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14155708#comment-14155708
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14155473#comment-14155473
]
Tsuyoshi OZAWA commented on YARN-2312:
--
I cannot reproduce the findbugs warning. Let m
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14151668#comment-14151668
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14146739#comment-14146739
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144806#comment-14144806
]
Jason Lowe commented on YARN-2312:
--
Took a brief look at the new patch. What I meant by m
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144538#comment-14144538
]
Hadoop QA commented on YARN-2312:
-
{color:red}-1 overall{color}. Here are the results of t
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144335#comment-14144335
]
Tsuyoshi OZAWA commented on YARN-2312:
--
Attached a first patch.
> Marking ContainerId
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144037#comment-14144037
]
Tsuyoshi OZAWA commented on YARN-2312:
--
Talked with Jian offline.
{quote}
2. Priority
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144033#comment-14144033
]
Tsuyoshi OZAWA commented on YARN-2312:
--
Thanks for your comment, Jason.
{quote}
We co
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14143699#comment-14143699
]
Jason Lowe commented on YARN-2312:
--
bq. One idea is to add id for upper 32 bits of contain
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14143158#comment-14143158
]
Tsuyoshi OZAWA commented on YARN-2312:
--
[~vinodkv], [~jianhe] do you have any feedback
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14138622#comment-14138622
]
Tsuyoshi OZAWA commented on YARN-2312:
--
Basically we can replace {{getId()}} with {{ge
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14136644#comment-14136644
]
Tsuyoshi OZAWA commented on YARN-2312:
--
I'll do this after YARN-2558.
> Marking Conta
[
https://issues.apache.org/jira/browse/YARN-2312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14136166#comment-14136166
]
Jian He commented on YARN-2312:
---
yep, we should update the caller of getId to use getContaine
32 matches
Mail list logo
