[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15074515#comment-15074515
]
Jun Gong commented on YARN-3480:
Thanks [~jianhe] for the suggestion, review and commit!
>
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15074459#comment-15074459
]
Hudson commented on YARN-3480:
--
FAILURE: Integrated in Hadoop-trunk-Commit #9036 (See
[https:
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15073567#comment-15073567
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15073461#comment-15073461
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15073360#comment-15073360
]
Jun Gong commented on YARN-3480:
[~jianhe] Thanks for suggestion. Attach a new patch to fix
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15073258#comment-15073258
]
Jian He commented on YARN-3480:
---
actually, one more comment:
below code introduces one more l
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15072438#comment-15072438
]
Jian He commented on YARN-3480:
---
lgtm, +1
> Recovery may get very slow with lots of service
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069147#comment-15069147
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069045#comment-15069045
]
Jun Gong commented on YARN-3480:
Discussed with [~jianhe] offline, we think the
implementa
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15068589#comment-15068589
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15068394#comment-15068394
]
Jun Gong commented on YARN-3480:
Attach a new patch, move remove attempts logic to RMStateS
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15067480#comment-15067480
]
Jun Gong commented on YARN-3480:
Thanks for explaining.
These cases make removing attempts
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15067143#comment-15067143
]
Jian He commented on YARN-3480:
---
Hi [~hex108], thanks for updating.
bq. If RMStateStore fails
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15066477#comment-15066477
]
Jun Gong commented on YARN-3480:
Test cases are not related, there are other issues trackin
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15066447#comment-15066447
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15066330#comment-15066330
]
Jun Gong commented on YARN-3480:
Attach a new patch to fix above problems. Considering
RM
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15065043#comment-15065043
]
Jun Gong commented on YARN-3480:
Thanks for review and suggestion!
{quote}
regarding this
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15064794#comment-15064794
]
Jian He commented on YARN-3480:
---
thanks for updating,
- regarding this logic, it is possible
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15061888#comment-15061888
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15061702#comment-15061702
]
Jun Gong commented on YARN-3480:
Thanks! The new patch addresses above problems and add tes
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15060978#comment-15060978
]
Jian He commented on YARN-3480:
---
thanks for updating, few more comments:
- rename startAttemp
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15060188#comment-15060188
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15059964#comment-15059964
]
Jun Gong commented on YARN-3480:
Fix findbugs and test errors.
> Recovery may get very slo
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15059828#comment-15059828
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15059689#comment-15059689
]
Jun Gong commented on YARN-3480:
Attach a new patch to address above problems.
> Recovery
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15059399#comment-15059399
]
Jun Gong commented on YARN-3480:
Thanks for the suggestion!
{quote}
I meant we can reuse t
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15059193#comment-15059193
]
Jian He commented on YARN-3480:
---
bq. In earlier patches, I did it in this way. Then max-allow
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15058076#comment-15058076
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057960#comment-15057960
]
Jun Gong commented on YARN-3480:
IMO, 'max-allowed-attempts' does not need be overridden to
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057754#comment-15057754
]
Jun Gong commented on YARN-3480:
Attach a new patch to add a global hard limit.
> Recovery
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057501#comment-15057501
]
Jun Gong commented on YARN-3480:
[~jianhe] Thanks for review and suggestion.
{quote}
how a
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15056844#comment-15056844
]
Jian He commented on YARN-3480:
---
[~hex108],
how about removing the attempts that are beyond
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15054819#comment-15054819
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15054341#comment-15054341
]
Hadoop QA commented on YARN-3480:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15054266#comment-15054266
]
Jun Gong commented on YARN-3480:
Hi [~jianhe], I just attached a patch that remove attempts
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15039613#comment-15039613
]
Jun Gong commented on YARN-3480:
[~jianhe] thanks for the remind. I thought the final solut
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15039560#comment-15039560
]
Jian He commented on YARN-3480:
---
[~hex108], i know it's been a long time, would you still lik
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14565385#comment-14565385
]
Vinod Kumar Vavilapalli commented on YARN-3480:
---
Tx. Linking YARN-1039.
> Re
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14553451#comment-14553451
]
Jun Gong commented on YARN-3480:
{quote}
Without doing this, we will unnecessarily be forci
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14547480#comment-14547480
]
Vinod Kumar Vavilapalli commented on YARN-3480:
---
bq. I think we need to have
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14547473#comment-14547473
]
Vinod Kumar Vavilapalli commented on YARN-3480:
---
bq. we might need keep faile
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14536112#comment-14536112
]
Jun Gong commented on YARN-3480:
[~vinodkv] Thanks for the suggestions.
{quote}
Part of wh
[
https://issues.apache.org/jira/browse/YARN-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14534761#comment-14534761
]
Vinod Kumar Vavilapalli commented on YARN-3480:
---
bq. RM HA is enabled, use ZK
43 matches
Mail list logo