[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14731149#comment-14731149 ] Hadoop QA commented on HADOOP-9382: --- \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | pre-patch | 20m 2s | Pre-patch trunk compilation is healthy. | | {color:green}+1{color} | @author | 0m 0s | The patch does not contain any @author tags. | | {color:green}+1{color} | tests included | 0m 0s | The patch appears to include 2 new or modified test files. | | {color:green}+1{color} | javac | 7m 38s | There were no new javac warning messages. | | {color:green}+1{color} | javadoc | 10m 2s | There were no new javadoc warning messages. | | {color:green}+1{color} | release audit | 0m 24s | The applied patch does not increase the total number of release audit warnings. | | {color:green}+1{color} | site | 3m 3s | Site still builds. | | {color:green}+1{color} | checkstyle | 1m 5s | There were no new checkstyle issues. | | {color:green}+1{color} | whitespace | 0m 1s | The patch has no lines that end in whitespace. | | {color:green}+1{color} | install | 1m 27s | mvn install still works. | | {color:green}+1{color} | eclipse:eclipse | 0m 31s | The patch built with eclipse:eclipse. | | {color:green}+1{color} | findbugs | 1m 53s | The patch does not introduce any new Findbugs (version 3.0.0) warnings. | | {color:red}-1{color} | common tests | 22m 56s | Tests failed in hadoop-common. | | | | 69m 6s | | \\ \\ || Reason || Tests || | Failed unit tests | hadoop.fs.TestLocalFsFCStatistics | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12754223/HADOOP-9382.4.patch | | Optional Tests | javadoc javac unit findbugs checkstyle site | | git revision | trunk / 6eaca2e | | hadoop-common test log | https://builds.apache.org/job/PreCommit-HADOOP-Build/7602/artifact/patchprocess/testrun_hadoop-common.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/7602/testReport/ | | Java | 1.7.0_55 | | uname | Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/7602/console | This message was automatically generated. > Add dfs mv overwrite option > --- > > Key: HADOOP-9382 > URL: https://issues.apache.org/jira/browse/HADOOP-9382 > Project: Hadoop Common > Issue Type: Improvement >Reporter: Keegan Witt >Assignee: Suresh Srinivas >Priority: Minor > Labels: BB2015-05-TBR > Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, > HADOOP-9382.3.patch, HADOOP-9382.4.patch, HADOOP-9382.patch > > > Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14731246#comment-14731246 ] Keegan Witt commented on HADOOP-9382: - Sorry for the multiple posts -- winutils is still giving my build issues. I think it's finally good to go. I'm pretty sure the test failure is unrelated to my changes, but please advise if you believe otherwise. > Add dfs mv overwrite option > --- > > Key: HADOOP-9382 > URL: https://issues.apache.org/jira/browse/HADOOP-9382 > Project: Hadoop Common > Issue Type: Improvement >Reporter: Keegan Witt >Assignee: Suresh Srinivas >Priority: Minor > Labels: BB2015-05-TBR > Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, > HADOOP-9382.3.patch, HADOOP-9382.4.patch, HADOOP-9382.patch > > > Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14730916#comment-14730916 ] Hadoop QA commented on HADOOP-9382: --- \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | pre-patch | 19m 57s | Pre-patch trunk compilation is healthy. | | {color:green}+1{color} | @author | 0m 0s | The patch does not contain any @author tags. | | {color:green}+1{color} | tests included | 0m 0s | The patch appears to include 2 new or modified test files. | | {color:green}+1{color} | javac | 7m 45s | There were no new javac warning messages. | | {color:green}+1{color} | javadoc | 9m 48s | There were no new javadoc warning messages. | | {color:green}+1{color} | release audit | 0m 23s | The applied patch does not increase the total number of release audit warnings. | | {color:green}+1{color} | site | 2m 59s | Site still builds. | | {color:red}-1{color} | checkstyle | 1m 4s | The applied patch generated 6 new checkstyle issues (total was 12, now 16). | | {color:red}-1{color} | whitespace | 0m 0s | The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix. | | {color:green}+1{color} | install | 1m 26s | mvn install still works. | | {color:green}+1{color} | eclipse:eclipse | 0m 31s | The patch built with eclipse:eclipse. | | {color:green}+1{color} | findbugs | 1m 51s | The patch does not introduce any new Findbugs (version 3.0.0) warnings. | | {color:red}-1{color} | common tests | 22m 9s | Tests failed in hadoop-common. | | | | 67m 58s | | \\ \\ || Reason || Tests || | Failed unit tests | hadoop.cli.TestCLI | | | hadoop.fs.TestFsShellCopy | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12754195/HADOOP-9382.3.patch | | Optional Tests | javadoc javac unit findbugs checkstyle site | | git revision | trunk / 40d222e | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/7601/artifact/patchprocess/diffcheckstylehadoop-common.txt | | whitespace | https://builds.apache.org/job/PreCommit-HADOOP-Build/7601/artifact/patchprocess/whitespace.txt | | hadoop-common test log | https://builds.apache.org/job/PreCommit-HADOOP-Build/7601/artifact/patchprocess/testrun_hadoop-common.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/7601/testReport/ | | Java | 1.7.0_55 | | uname | Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/7601/console | This message was automatically generated. > Add dfs mv overwrite option > --- > > Key: HADOOP-9382 > URL: https://issues.apache.org/jira/browse/HADOOP-9382 > Project: Hadoop Common > Issue Type: Improvement >Reporter: Keegan Witt >Assignee: Suresh Srinivas >Priority: Minor > Labels: BB2015-05-TBR > Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, > HADOOP-9382.3.patch, HADOOP-9382.patch > > > Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14712559#comment-14712559 ] Gautam Gopalakrishnan commented on HADOOP-9382: --- [~keegan] please submit the patch again on trunk, the latest one in the jira is over two years old and doesn't apply cleanly. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Labels: BB2015-05-TBR Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14711221#comment-14711221 ] Keegan Witt commented on HADOOP-9382: - Since Unix disallows replacing a file with a directory, and calling {{Filesystem.rename()}} directly fails under this circumstance, I'd expect the shell invocation to fail as well. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Labels: BB2015-05-TBR Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14710510#comment-14710510 ] Gautam Gopalakrishnan commented on HADOOP-9382: --- [~keegan] Could you respond to Colin's questions please, we can work on getting this feature committed in the next few weeks. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Labels: BB2015-05-TBR Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654154#comment-14654154 ] Colin Patrick McCabe commented on HADOOP-9382: -- What is the behavior supposed to be when the source is a directory, and the destination is a file? In UNIX, a directory cannot be overwritten by a file. {code} cmccabe@keter:~ mkdir d cmccabe@keter:~ touch f cmccabe@keter:~ mv d f mv: overwrite ‘f’? y mv: cannot overwrite non-directory ‘f’ with directory ‘d’ cmccabe@keter:~ mv -f c b mv: cannot overwrite non-directory ‘f’ with directory ‘d’ {code} Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Labels: BB2015-05-TBR Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654165#comment-14654165 ] Hadoop QA commented on HADOOP-9382: --- \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:red}-1{color} | patch | 0m 0s | The patch command could not apply the patch during dryrun. | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch | | Optional Tests | javadoc javac unit findbugs checkstyle site | | git revision | trunk / 9a08999 | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/7408/console | This message was automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Labels: BB2015-05-TBR Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14526836#comment-14526836 ] Keegan Witt commented on HADOOP-9382: - Not sure what triggered the last build...I'd given up hope of this patch being applied. If someone is willing to look at this, I'm willing to rebase again. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14525035#comment-14525035 ] Hadoop QA commented on HADOOP-9382: --- \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:red}-1{color} | patch | 0m 0s | The patch command could not apply the patch during dryrun. | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch | | Optional Tests | javadoc javac unit findbugs checkstyle site | | git revision | trunk / f1a152c | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/6378/console | This message was automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14524980#comment-14524980 ] Hadoop QA commented on HADOOP-9382: --- \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:red}-1{color} | patch | 0m 0s | The patch command could not apply the patch during dryrun. | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch | | Optional Tests | javadoc javac unit findbugs checkstyle site | | git revision | trunk / f1a152c | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/6370/console | This message was automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14185148#comment-14185148 ] Hadoop QA commented on HADOOP-9382: --- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch against trunk revision 0058ead. {color:red}-1 patch{color}. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/4959//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14075802#comment-14075802 ] Hadoop QA commented on HADOOP-9382: --- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch against trunk revision . {color:red}-1 patch{color}. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/4369//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Suresh Srinivas Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13911216#comment-13911216 ] Keegan Witt commented on HADOOP-9382: - [~sureshms], have you had a chance to look at this patch? Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.1.5#6160)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13911248#comment-13911248 ] Hadoop QA commented on HADOOP-9382: --- {color:green}+1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch against trunk revision . {color:green}+1 @author{color}. The patch does not contain any @author tags. {color:green}+1 tests included{color}. The patch appears to include 2 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 1.3.9) warnings. {color:green}+1 release audit{color}. The applied patch does not increase the total number of release audit warnings. {color:green}+1 core tests{color}. The patch passed unit tests in hadoop-common-project/hadoop-common. {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/3604//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3604//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message was sent by Atlassian JIRA (v6.1.5#6160)
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13759292#comment-13759292 ] Keegan Witt commented on HADOOP-9382: - [~sureshms], have you had a chance to look over my latest patch? Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13759352#comment-13759352 ] Suresh Srinivas commented on HADOOP-9382: - [~keegan], sorry I have been busy. Will look into this in next couple of days. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13740773#comment-13740773 ] Hadoop QA commented on HADOOP-9382: --- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598152/HADOOP-9382.1.patch against trunk revision . {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}. The javadoc tool did not generate any 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 1.3.9) 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-common-project/hadoop-common: org.apache.hadoop.cli.TestCLI {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2985//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2985//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13740960#comment-13740960 ] Suresh Srinivas commented on HADOOP-9382: - Comments: # Please undo unnecessary empty line changes, such as the following: {code} } - + protected PathData getTargetPath(PathData src) throws IOException { {code} TestCLI failure is related to this change that needs to be fixed. You can run a specific test by: * {{cd to hadoop-common-project/hadoop-common}} * {{mvn -Dtest=TestCLI test}} Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13740986#comment-13740986 ] Hadoop QA commented on HADOOP-9382: --- {color:green}+1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598195/HADOOP-9382.2.patch against trunk revision . {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}. The javadoc tool did not generate any 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 1.3.9) warnings. {color:green}+1 release audit{color}. The applied patch does not increase the total number of release audit warnings. {color:green}+1 core tests{color}. The patch passed unit tests in hadoop-common-project/hadoop-common. {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2987//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2987//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13741004#comment-13741004 ] Keegan Witt commented on HADOOP-9382: - OK, I took those out. There are a couple of Javadoc errors that were in the patch that someone should look into at some point (I took those out too). Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13741072#comment-13741072 ] Hadoop QA commented on HADOOP-9382: --- {color:green}+1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12598214/HADOOP-9382.2.patch against trunk revision . {color:green}+1 @author{color}. The patch does not contain any @author tags. {color:green}+1 tests included{color}. The patch appears to include 2 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}. The javadoc tool did not generate any 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 1.3.9) warnings. {color:green}+1 release audit{color}. The applied patch does not increase the total number of release audit warnings. {color:green}+1 core tests{color}. The patch passed unit tests in hadoop-common-project/hadoop-common. {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2988//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2988//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13741249#comment-13741249 ] Keegan Witt commented on HADOOP-9382: - By in the patch, I meant that I fixed in the previous patch. This latest patch should be good to go. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Assignee: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.2.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13740712#comment-13740712 ] Keegan Witt commented on HADOOP-9382: - I think I've resolved the issues with the tests. I've not been able to run a full mvn test, as I'm having environment issues again. But I ran the test files I updated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382.1.patch, HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13599693#comment-13599693 ] Keegan Witt commented on HADOOP-9382: - I finally got the compile working. The patch includes a unit test. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13599712#comment-13599712 ] Hadoop QA commented on HADOOP-9382: --- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12573255/HADOOP-9382.patch against trunk revision . {color:green}+1 @author{color}. The patch does not contain any @author tags. {color:green}+1 tests included{color}. The patch appears to include 1 new or modified test files. {color:red}-1 one of tests included doesn't have a timeout.{color} {color:green}+1 javac{color}. The applied patch does not increase the total number of javac compiler warnings. {color:green}+1 javadoc{color}. The javadoc tool did not generate any 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 1.3.9) 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-common-project/hadoop-common: org.apache.hadoop.fs.TestFsShellCopy org.apache.hadoop.cli.TestCLI {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2314//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2314//console This message is automatically generated. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Affects Versions: 2.0.0-alpha Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382.patch Add a -f option to allow overwriting existing destinations in dfs mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13597386#comment-13597386 ] Keegan Witt commented on HADOOP-9382: - Actually, this should also apply to moveFromLocal. I'll update the patch. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch, HADOOP-9382_2.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13596298#comment-13596298 ] Keegan Witt commented on HADOOP-9382: - Don't forget to update the latest web [documentation page|http://hadoop.apache.org/docs/r1.1.2/file_system_shell.html] (I don't know where the source for that is to include a patch). Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13596378#comment-13596378 ] Suresh Srinivas commented on HADOOP-9382: - bq. (I don't know where the source for that is to include a patch). Please see {{hadoop-common-project/hadoop-common/src/site/apt/FileSystemShell.apt.vm}}. Please add unit tests to ensure this functionality works. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13596871#comment-13596871 ] Keegan Witt commented on HADOOP-9382: - Ah, thank you. I'll work on tests for this (it'll take a little bit because it doesn't look like there's currently any tests for this class atm). Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13596873#comment-13596873 ] Keegan Witt commented on HADOOP-9382: - Oh, one thing I noticed while trying to work on the test for this is that I can't compile because Maven can't find the classes in RpcHeaderProtos. Do you know what I might be doing wrong? I couldn't seem to find any documentation on this. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9382) Add dfs mv overwrite option
[ https://issues.apache.org/jira/browse/HADOOP-9382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13596878#comment-13596878 ] Suresh Srinivas commented on HADOOP-9382: - This might be related to you not having protoc for compiling protobuf stuff? Send me an email about error msg. Add dfs mv overwrite option --- Key: HADOOP-9382 URL: https://issues.apache.org/jira/browse/HADOOP-9382 Project: Hadoop Common Issue Type: Improvement Reporter: Keegan Witt Priority: Minor Attachments: HADOOP-9382_1.patch Add a -f option to allow overwriting existing destinations in mv command. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira