[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236416#comment-16236416
]
Hadoop QA commented on YARN-7370:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236281#comment-16236281
]
Hudson commented on YARN-7370:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13180 (See
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236155#comment-16236155
]
Gergely Novák commented on YARN-7370:
-
Sorry, I already did (patch #5).
> Preemption properties should
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236151#comment-16236151
]
Eric Payne commented on YARN-7370:
--
Findbugs warnings are for
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236106#comment-16236106
]
Hadoop QA commented on YARN-7370:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236089#comment-16236089
]
Eric Payne commented on YARN-7370:
--
Waiting for precommit to complete:
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16236040#comment-16236040
]
Sunil G commented on YARN-7370:
---
[~eepayne] Latest patch is fine to me. Pls help to commit.
> Preemption
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235898#comment-16235898
]
Sunil G commented on YARN-7370:
---
Thanks [~GergelyNovak]
> Preemption properties should be refreshable
>
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235888#comment-16235888
]
Subru Krishnan commented on YARN-7370:
--
[~eepayne], please go ahead and cherry-pick to branch-2.9. FYI
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235880#comment-16235880
]
Gergely Novák commented on YARN-7370:
-
Thanks [~sunilg] for the suggestion, I removed the null check in
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235862#comment-16235862
]
Eric Payne commented on YARN-7370:
--
[~sunilg], thanks for your comment. I see what you mean. If we take
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235826#comment-16235826
]
Sunil G commented on YARN-7370:
---
Sorry for jumping in little late here. Thanks for the effort [~GergelyNovak]
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235817#comment-16235817
]
Eric Payne commented on YARN-7370:
--
[~GergelyNovak], Thanks for your effort on this feature. Patch LGTM
+1
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235747#comment-16235747
]
Eric Payne commented on YARN-7370:
--
[~subru] and [~asuresh],
I am aware that you have requested that
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235740#comment-16235740
]
Eric Payne commented on YARN-7370:
--
Thanks for the patch, [~GergelyNovak]. I will review today.
bq. The
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16235613#comment-16235613
]
Gergely Novák commented on YARN-7370:
-
Added the above 4 properties to the log statement in patch #3.
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16227276#comment-16227276
]
Eric Payne commented on YARN-7370:
--
[~GergelyNovak], Thanks for the updated patch. Just a couple of
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16224824#comment-16224824
]
Gergely Novák commented on YARN-7370:
-
[~eepayne] Thank you for your suggestions, addressed them in
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16222830#comment-16222830
]
Eric Payne commented on YARN-7370:
--
Thanks [~GergelyNovak] for the work on this patch. I just have a
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16219035#comment-16219035
]
Gergely Novák commented on YARN-7370:
-
[~eepayne] He must have meant YARN-6124, not YARN-6142.
I
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16218617#comment-16218617
]
Eric Payne commented on YARN-7370:
--
Thanks [~leftnoteasy] for the further design specifications.
bq.
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16217476#comment-16217476
]
Wangda Tan commented on YARN-7370:
--
Link this JIRA to YARN-6124.
> Preemption properties should be
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16217478#comment-16217478
]
Wangda Tan commented on YARN-7370:
--
Offline discussing this JIRA with [~GergelyNovak], I felt it might be
[
https://issues.apache.org/jira/browse/YARN-7370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16216007#comment-16216007
]
Wangda Tan commented on YARN-7370:
--
Make sense, [~eepayne].
I just found previously I filed a JIRA:
24 matches
Mail list logo