[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15693855#comment-15693855
]
Hadoop QA commented on HADOOP-13018:
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15693810#comment-15693810
]
Hudson commented on HADOOP-13018:
-
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10888 (See
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15693708#comment-15693708
]
Steve Loughran commented on HADOOP-13018:
-
LTGM
+1
> Make Kdiag check whether
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15398415#comment-15398415
]
Hadoop QA commented on HADOOP-13018:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15398340#comment-15398340
]
Ravi Prakash commented on HADOOP-13018:
---
Hi Steve!
I'm not sure what to add to the document. The
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15272150#comment-15272150
]
Steve Loughran commented on HADOOP-13018:
-
# I agree with your sentiments on the checkstyle, but
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15271448#comment-15271448
]
Hadoop QA commented on HADOOP-13018:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15270501#comment-15270501
]
Steve Loughran commented on HADOOP-13018:
-
* checkstyle is fussing about indentation and a
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15269565#comment-15269565
]
Hadoop QA commented on HADOOP-13018:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15259832#comment-15259832
]
Steve Loughran commented on HADOOP-13018:
-
lets do the specific one, as it catches anything
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15259649#comment-15259649
]
Ravi Prakash commented on HADOOP-13018:
---
Do you mean this instead?
{code} if
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15257756#comment-15257756
]
Steve Loughran commented on HADOOP-13018:
-
I'm just trying to understand why you have to go
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15257047#comment-15257047
]
Ravi Prakash commented on HADOOP-13018:
---
Hi Steve! I felt {{verify}} shouldn't be Method specific.
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15256133#comment-15256133
]
Steve Loughran commented on HADOOP-13018:
-
Looks great!
One question: why the reflection?
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15255492#comment-15255492
]
Hadoop QA commented on HADOOP-13018:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15253598#comment-15253598
]
Steve Loughran commented on HADOOP-13018:
-
# you can use {{verifyFileIsValid}} is to validate
[
https://issues.apache.org/jira/browse/HADOOP-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15253295#comment-15253295
]
Hadoop QA commented on HADOOP-13018:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
17 matches
Mail list logo