[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[ https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14302800#comment-14302800 ] Tsuyoshi OZAWA commented on HADOOP-11043: - Cancelling a patch. Use Path Instead of String in DistCp Tests -- Key: HADOOP-11043 URL: https://issues.apache.org/jira/browse/HADOOP-11043 Project: Hadoop Common Issue Type: Improvement Components: tools/distcp Reporter: Gary Steelman Assignee: Gary Steelman Priority: Minor Attachments: HADOOP-11043.1.patch Many DistCp tests mix String and Path for files for no real reason. This patch aligns use to Path. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[ https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14294987#comment-14294987 ] Tsuyoshi OZAWA commented on HADOOP-11043: - This test failure looks related to the attached patch. [~gsteelman], could you update to fix the test failure? Use Path Instead of String in DistCp Tests -- Key: HADOOP-11043 URL: https://issues.apache.org/jira/browse/HADOOP-11043 Project: Hadoop Common Issue Type: Improvement Components: tools/distcp Reporter: Gary Steelman Assignee: Gary Steelman Priority: Minor Attachments: HADOOP-11043.1.patch Many DistCp tests mix String and Path for files for no real reason. This patch aligns use to Path. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[
https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14294975#comment-14294975
]
Hadoop QA commented on HADOOP-11043:
{color:red}-1 overall{color}. Here are the results of testing the latest
attachment
http://issues.apache.org/jira/secure/attachment/12665806/HADOOP-11043.1.patch
against trunk revision d12dd47.
{color:green}+1 @author{color}. The patch does not contain any @author
tags.
{color:green}+1 tests included{color}. The patch appears to include 3 new
or modified test files.
{color:green}+1 javac{color}. The applied patch does not increase the
total number of javac compiler warnings.
{color:green}+1 javadoc{color}. There were no new javadoc warning messages.
{color:green}+1 eclipse:eclipse{color}. The patch built with
eclipse:eclipse.
{color:green}+1 findbugs{color}. The patch does not introduce any new
Findbugs (version 2.0.3) warnings.
{color:green}+1 release audit{color}. The applied patch does not increase
the total number of release audit warnings.
{color:red}-1 core tests{color}. The patch failed these unit tests in
hadoop-tools/hadoop-distcp:
org.apache.hadoop.tools.mapred.TestCopyCommitter
Test results:
https://builds.apache.org/job/PreCommit-HADOOP-Build/5516//testReport/
Console output:
https://builds.apache.org/job/PreCommit-HADOOP-Build/5516//console
This message is automatically generated.
Use Path Instead of String in DistCp Tests
--
Key: HADOOP-11043
URL: https://issues.apache.org/jira/browse/HADOOP-11043
Project: Hadoop Common
Issue Type: Improvement
Components: tools/distcp
Reporter: Gary Steelman
Assignee: Gary Steelman
Priority: Minor
Attachments: HADOOP-11043.1.patch
Many DistCp tests mix String and Path for files for no real reason. This
patch aligns use to Path.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[ https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14117672#comment-14117672 ] Gary Steelman commented on HADOOP-11043: The entire test suite for TestDistCpUtils was expanded and refactored in HADOOP-11009. This patch requires that patch to be applied first. Use Path Instead of String in DistCp Tests -- Key: HADOOP-11043 URL: https://issues.apache.org/jira/browse/HADOOP-11043 Project: Hadoop Common Issue Type: Improvement Components: tools/distcp Reporter: Gary Steelman Priority: Minor Attachments: HADOOP-11043.1.patch Many DistCp tests mix String and Path for files for no real reason. This patch aligns use to Path. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[
https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14117675#comment-14117675
]
Hadoop QA commented on HADOOP-11043:
{color:red}-1 overall{color}. Here are the results of testing the latest
attachment
http://issues.apache.org/jira/secure/attachment/12665806/HADOOP-11043.1.patch
against trunk revision 258c7d0.
{color:red}-1 patch{color}. The patch command could not apply the patch.
Console output:
https://builds.apache.org/job/PreCommit-HADOOP-Build/4622//console
This message is automatically generated.
Use Path Instead of String in DistCp Tests
--
Key: HADOOP-11043
URL: https://issues.apache.org/jira/browse/HADOOP-11043
Project: Hadoop Common
Issue Type: Improvement
Components: tools/distcp
Reporter: Gary Steelman
Priority: Minor
Attachments: HADOOP-11043.1.patch
Many DistCp tests mix String and Path for files for no real reason. This
patch aligns use to Path.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[ https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14117815#comment-14117815 ] Tsuyoshi OZAWA commented on HADOOP-11043: - Thanks for your contribution, Gary. Let's make this JIRA patch available after HADOOP-11009 will be done. Use Path Instead of String in DistCp Tests -- Key: HADOOP-11043 URL: https://issues.apache.org/jira/browse/HADOOP-11043 Project: Hadoop Common Issue Type: Improvement Components: tools/distcp Reporter: Gary Steelman Priority: Minor Attachments: HADOOP-11043.1.patch Many DistCp tests mix String and Path for files for no real reason. This patch aligns use to Path. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-11043) Use Path Instead of String in DistCp Tests
[ https://issues.apache.org/jira/browse/HADOOP-11043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14117860#comment-14117860 ] Gary Steelman commented on HADOOP-11043: Thanks again [~ozawa]. I wasn't sure how to mark a patch as waiting on another patch to be accepted, but once it is, this one should work. Use Path Instead of String in DistCp Tests -- Key: HADOOP-11043 URL: https://issues.apache.org/jira/browse/HADOOP-11043 Project: Hadoop Common Issue Type: Improvement Components: tools/distcp Reporter: Gary Steelman Priority: Minor Attachments: HADOOP-11043.1.patch Many DistCp tests mix String and Path for files for no real reason. This patch aligns use to Path. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
