[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16333796#comment-16333796
]
Genmao Yu commented on HADOOP-15039:
[~Sammi] got it
> Move SemaphoredDelegatingExecutor to
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16331974#comment-16331974
]
SammiChen commented on HADOOP-15039:
Hi [~uncleGen], HADOOP-15027 is depend on this JIRA. I would
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16290291#comment-16290291
]
Hudson commented on HADOOP-15039:
-
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13372 (See
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16290282#comment-16290282
]
Kai Zheng commented on HADOOP-15039:
Just done. The commit was corrected.
Sorry for the
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16290276#comment-16290276
]
Kai Zheng commented on HADOOP-15039:
Oops, the commit was incorrect. Thanks [~uncleGen] for the
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16279644#comment-16279644
]
Hudson commented on HADOOP-15039:
-
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13332 (See
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16279626#comment-16279626
]
Kai Zheng commented on HADOOP-15039:
The latest patch LGTM and +1. Thanks Genmao for the update!
>
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16279601#comment-16279601
]
Genmao Yu commented on HADOOP-15039:
[~drankye] Take a review again please!
> Move
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16279586#comment-16279586
]
genericqa commented on HADOOP-15039:
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278609#comment-16278609
]
genericqa commented on HADOOP-15039:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278412#comment-16278412
]
genericqa commented on HADOOP-15039:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278162#comment-16278162
]
genericqa commented on HADOOP-15039:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276251#comment-16276251
]
Kai Zheng commented on HADOOP-15039:
Thanks [~uncleGen] for the work. It LGTM and the minor comment
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16272132#comment-16272132
]
Genmao Yu commented on HADOOP-15039:
[~drankye] Could you please keep an eye on it. IMHO, this patch
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16267909#comment-16267909
]
Genmao Yu commented on HADOOP-15039:
[~ste...@apache.org] Could you take a review again please?
>
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16262756#comment-16262756
]
Genmao Yu commented on HADOOP-15039:
[~ste...@apache.org] take a look please.
> move
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16260224#comment-16260224
]
Hadoop QA commented on HADOOP-15039:
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16260170#comment-16260170
]
Genmao Yu commented on HADOOP-15039:
make sense and add them.
> move SemaphoredDelegatingExecutor
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16259654#comment-16259654
]
Steve Loughran commented on HADOOP-15039:
-
Looks good, but we need those
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16259062#comment-16259062
]
Genmao Yu commented on HADOOP-15039:
[~ste...@apache.org] all unit test/integration test passed
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16257264#comment-16257264
]
Steve Loughran commented on HADOOP-15039:
-
You do I'm afraid, it's fairly straightforward and
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16256379#comment-16256379
]
Genmao Yu commented on HADOOP-15039:
[~ste...@apache.org] Looks like this patch change is clean, do
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16255225#comment-16255225
]
Hadoop QA commented on HADOOP-15039:
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16255097#comment-16255097
]
Hadoop QA commented on HADOOP-15039:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16254972#comment-16254972
]
Genmao Yu commented on HADOOP-15039:
move class {{SemaphoredDelegatingExecutor}} and
25 matches
Mail list logo