[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15074435#comment-15074435
]
Rich Haase commented on YARN-4257:
--
[~leftnoteasy] Agreed. Doesn't make sense to change validateConf if
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15074404#comment-15074404
]
Rich Haase commented on YARN-4257:
--
[~kasha] I can update the patch to revert the changes to the
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15073189#comment-15073189
]
Rich Haase commented on YARN-4257:
--
The failing tests don't appear to be related to this patch. Not sure
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15073053#comment-15073053
]
Rich Haase commented on YARN-4257:
--
Just uploaded the fixed patch. I found a couple import changes that
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: YARN-4257.patch
> Move scheduler validateConf method to AbstractYarnScheduler and make it
>
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: (was: YARN-4257.patch)
> Move scheduler validateConf method to AbstractYarnScheduler and
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15073003#comment-15073003
]
Rich Haase commented on YARN-4257:
--
Hi [~wangda],
I'm uploading a patch that tests out in my local build
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: (was: YARN-4257.patch)
> Move scheduler validateConf method to AbstractYarnScheduler and
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: YARN-4257.patch
Uploading corrected patch. My apologies for the previous incorrect diff
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: YARN-4257.patch
> Move scheduler validateConf method to AbstractYarnScheduler and make it
>
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: (was: YARN-4257.patch)
> Move scheduler validateConf method to AbstractYarnScheduler and
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15062558#comment-15062558
]
Rich Haase commented on YARN-4257:
--
Reviewing failed tests.
> Move scheduler validateConf method to
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15062590#comment-15062590
]
Rich Haase commented on YARN-4257:
--
I've uploaded a new patch that removes the failing tests for zero MB
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase reassigned YARN-4257:
Assignee: Rich Haase
> Move scheduler validateConf method to AbstractYarnScheduler and make it
>
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15060808#comment-15060808
]
Rich Haase commented on YARN-4257:
--
Makes sense to me.
> Move scheduler validateConf method to
[
https://issues.apache.org/jira/browse/YARN-4257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4257:
-
Attachment: YARN-4257.patch
> Move scheduler validateConf method to AbstractYarnScheduler and make it
>
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4207:
-
Attachment: (was: YARN-4207.patch)
> Add a non-judgemental YARN app completion status
>
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059460#comment-15059460
]
Rich Haase commented on YARN-4207:
--
Justification for not adding a test: adding a value to an enum is to
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4207:
-
Attachment: (was: HADOOP-12085.patch)
> Add a non-judgemental YARN app completion status
>
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4207:
-
Attachment: YARN-4207.patch
> Add a non-judgemental YARN app completion status
>
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4207:
-
Attachment: HADOOP-12085.patch
Sorry for the slow response all. I've update the patch to include the new
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Haase updated YARN-4207:
-
Attachment: YARN-4207.patch
Trivial patch that adds ENDED as a YARN application status.
> Add a
[
https://issues.apache.org/jira/browse/YARN-4207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14949546#comment-14949546
]
Rich Haase commented on YARN-4207:
--
This looks like a pretty trivial change. Adding an additional value
23 matches
Mail list logo
