[Hadoop-common-dev] [jira] Created: (HADOOP-3100) Develop tests to test the DFS command line interface

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12582495#action_12582495 ]

Mahadev konar commented on HADOOP-3100:
---------------------------------------

most of these are tested in TestDFSShell .... testErrOut() method.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Attachment: H3100.patch

Initial patch with the test driver code and test cases for:
ls
lsr
du
dus
mv

There are 31 test cases which perform 100 validations

The test cases (input commands and expected output) are defined in the testConf.xml file. This is the file that will be updated with other test cases.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Status: Patch Available (was: Open)

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Attachment: H3100.patch

Here is an updated patch. It includes changes to the test driver and additional tests. Thanks to Karam and Ramya for the additional tests. There are 139 tests with 283 validations being done.

The tests have been run on Windows, Mac and Linux. It adds < 30 seconds on linux/mac and 3 mins to the unit tests runtime on windows.

I have run the ant test-patch target and there were no errors reported.
[exec] +1 overall.

[exec] @author +1. The patch does not contain any @author tags.

[exec] tests included +1. The patch appears to include 323 new or modified tests.

[exec] javadoc +1. The javadoc tool did not generate any warning messages.

[exec] javac +1. The applied patch does not generate any new javac compiler warnings.

[exec] findbugs +1. The patch does not introduce any new Findbugs warnings.



Here are test cases being covered and the output from the test:

2008-04-30 09:45:55,298 INFO cli.TestCLI (TestCLI.java:displayResults(170)) - Detailed results:
2008-04-30 09:45:55,299 INFO cli.TestCLI (TestCLI.java:displayResults(171)) - ----------------------------------

2008-04-30 09:45:55,299 INFO cli.TestCLI (TestCLI.java:displayResults(215)) - Summary results:
2008-04-30 09:45:55,299 INFO cli.TestCLI (TestCLI.java:displayResults(216)) - ----------------------------------

2008-04-30 09:45:55,300 INFO cli.TestCLI (TestCLI.java:displayResults(236)) - Testing mode: test
2008-04-30 09:45:55,300 INFO cli.TestCLI (TestCLI.java:displayResults(237)) -
2008-04-30 09:45:55,300 INFO cli.TestCLI (TestCLI.java:displayResults(238)) - Overall result: +++ PASS +++
2008-04-30 09:45:55,300 INFO cli.TestCLI (TestCLI.java:displayResults(240)) - # Tests pass: 139 (100%)
2008-04-30 09:45:55,303 INFO cli.TestCLI (TestCLI.java:displayResults(242)) - # Tests fail: 0 (0%)
2008-04-30 09:45:55,304 INFO cli.TestCLI (TestCLI.java:displayResults(244)) - # Validations done: 283 (each test may do multiple validations)
2008-04-30 09:45:55,304 INFO cli.TestCLI (TestCLI.java:displayResults(247)) -
2008-04-30 09:45:55,304 INFO cli.TestCLI (TestCLI.java:displayResults(248)) - Failing tests:
2008-04-30 09:45:55,305 INFO cli.TestCLI (TestCLI.java:displayResults(249)) - --------------
2008-04-30 09:45:55,305 INFO cli.TestCLI (TestCLI.java:displayResults(261)) - NONE
2008-04-30 09:45:55,305 INFO cli.TestCLI (TestCLI.java:displayResults(265)) -
2008-04-30 09:45:55,306 INFO cli.TestCLI (TestCLI.java:displayResults(266)) - Passing tests:
2008-04-30 09:45:55,306 INFO cli.TestCLI (TestCLI.java:displayResults(267)) - --------------
2008-04-30 09:45:55,306 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 1: ls: file using absolute path
2008-04-30 09:45:55,306 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 2: ls: file using relative path
2008-04-30 09:45:55,306 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 3: ls: files using globbing
2008-04-30 09:45:55,307 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 4: ls: directory using absolute path
2008-04-30 09:45:55,307 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 5: ls: directory using relative path
2008-04-30 09:45:55,307 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 6: ls: directory using globbing
2008-04-30 09:45:55,307 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 7: ls: file/directory that does not exist in /
2008-04-30 09:45:55,307 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 8: ls: file/directory that does not exist in home directory (/user/username)
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 9: lsr: files/directories using absolute path
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 10: lsr: files/directories using relative path
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 11: lsr: files/directories using globbing
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 12: lsr: file/directory that does not exist in /
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 13: lsr: file/directory that does not exist in home directory (/user/username)
2008-04-30 09:45:55,308 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 14: du: file using absolute path
2008-04-30 09:45:55,309 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 15: du: file using relative path
2008-04-30 09:45:55,313 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 16: du: files using globbing
2008-04-30 09:45:55,314 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 17: du: directory using absolute path
2008-04-30 09:45:55,314 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 18: du: directory using relative path
2008-04-30 09:45:55,314 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 19: du: directory using globbing
2008-04-30 09:45:55,314 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 20: dus: directories/files using absolute path
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 21: dus: directories/files using relative path
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 22: dus: directories/files using globbing
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 23: mv: file (absolute path) to file (absolute path)
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 24: mv: file (absolute path) to file (relative path)
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 25: mv: file (absolute path) to directory (absolute path); keep the same name at the destination
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 26: mv: file (absolute path) to directory (absolute path); keep the same name at the destination [ TIED to previous test ]
2008-04-30 09:45:55,315 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 27: mv: file (absolute path) to directory (absolute path); change the name at the destination
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 28: mv: file (absolute path) to directory (absolute path); change the name at the destination [ TIED to previous test ]
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 29: mv: files (absolute path) to directory (absolute path) using globbing
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 30: mv: files (absolute path) to directory (absolute path) using globbing [ TIED to previous test ]
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 31: mv: file (relative) to file (relative)
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 32: cp: file (absolute path) to file (absolute path)
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 33: cp: file (absolute path) to file (relative path)
2008-04-30 09:45:55,316 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 34: cp: file (relative path) to file (absolute path)
2008-04-30 09:45:55,317 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 35: cp: file (relative path) to file (relative path)
2008-04-30 09:45:55,317 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 36: cp: file (absolute path) to directory (absolute path); keep the same name at the destination
2008-04-30 09:45:55,317 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 37: cp: file (absolute path) to directory (absolute path); change the name at the destination
2008-04-30 09:45:55,317 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 38: cp: files to directory (absolute path) using globbing
2008-04-30 09:45:55,317 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 39: cp: files to directory (absolute path) without globbing
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 40: cp: copying non existent file (absolute path)
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 41: cp: copying non existent file (relative path)
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 42: cp: files to an existent file using globbing
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 43: cp: files to an existent file without globbing
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 44: cp: files to a non existent directory using globbing
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 45: cp: files to a non existent directory without globbing
2008-04-30 09:45:55,318 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 46: rm: removing a file (absolute path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 47: rm: removing a file (relative path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 48: rm: removing files by globbing (absolute path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 49: rm: removing files by globbing (relative path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 50: rm: removing a directory (absolute path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 51: rm: removing a directory (relative path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 52: rm: removing a nonexistent file (absolute path)
2008-04-30 09:45:55,319 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 53: rm: removing a nonexistent file (relative path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 54: rmr: removing a file (absolute path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 55: rmr: removing a file (relative path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 56: rmr: removing a directory (absolute path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 57: rmr: removing a directory (relative path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 58: rmr: removing directories by globbing (absolute path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 59: rmr: removing directories by globbing (relative path)
2008-04-30 09:45:55,320 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 60: rmr: removing a nonexistent file (absolute path)
2008-04-30 09:45:55,321 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 61: rmr: removing a nonexistent file (relative path)
2008-04-30 09:45:55,321 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 62: put: putting file into a file (absolute path)
2008-04-30 09:45:55,321 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 63: put: putting file into a file (relative path)
2008-04-30 09:45:55,321 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 64: put: putting file into a directory(absolute path)
2008-04-30 09:45:55,321 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 65: put: putting file into a directory(relative path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 66: put: putting many files into an existing directory(absolute path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 67: put: putting many files into an existing directory(relative path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 68: put: putting non existent file(absolute path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 69: put: putting non existent file(relative path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 70: put: putting file into an already existing destination(absolute path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 71: put: putting file into an already existing destination(relative path)
2008-04-30 09:45:55,322 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 72: put: putting many files into an existing file
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 73: put: putting many files into a non existent directory
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 74: copyFromLocal: copying file into a file (absolute path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 75: copyFromLocal: copying file into a file (relative path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 76: copyFromLocal: copying file into a directory(absolute path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 77: copyFromLocal: copying file into a directory(relative path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 78: copyFromLocal: copying many files into an existing directory(absolute path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 79: copyFromLocal: copying many files into an existing directory(relative path)
2008-04-30 09:45:55,323 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 80: copyFromLocal: copying non existent file(absolute path)
2008-04-30 09:45:55,324 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 81: copyFromLocal: copying non existent file(relative path)
2008-04-30 09:45:55,324 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 82: copyFromLocal: copying file into an already existing destination(absolute path)
2008-04-30 09:45:55,324 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 83: copyFromLocal: copying file into an already existing destination(relative path)
2008-04-30 09:45:55,324 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 84: copyFromLocal: copying many files into an existing file
2008-04-30 09:45:55,324 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 85: copyFromLocal: copying many files into a non existent directory
2008-04-30 09:45:55,325 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 86: get: getting non existent(absolute path)
2008-04-30 09:45:55,325 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 87: get: getting non existent file(relative path)
2008-04-30 09:45:55,325 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 88: cat: contents of file(absolute path)
2008-04-30 09:45:55,325 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 89: cat: contents of file(relative path)
2008-04-30 09:45:55,325 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 90: cat: contents of files(absolute path) using globbing
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 91: cat: contents of files(relative path) using globbing
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 92: cat: contents of files(absolute path) without globbing
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 93: cat: contents of files(relative path) without globbing
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 94: cat: contents of file(absolute path) that does not exist
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 95: cat: contents of file(relative path) that does not exist
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 96: cat: contents of directory(absolute path)
2008-04-30 09:45:55,326 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 97: cat: contents of directory(relative path)
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 98: copyToLocal: non existent relative path
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 99: copyToLocal: non existent absolute path
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 100: mkdir: creating directory (absolute path)
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 101: mkdir: creating directory (relative path)
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 102: mkdir: creating many directories (absolute path)
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 103: mkdir: creating many directories (relative path)
2008-04-30 09:45:55,327 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 104: mkdir: creating a directory with the name of an already existing directory
2008-04-30 09:45:55,328 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 105: mkdir: creating a directory with the name of an already existing file
2008-04-30 09:45:55,328 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 106: setrep: existent file (absolute path)
2008-04-30 09:45:55,328 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 107: setrep: existent file (relative path)
2008-04-30 09:45:55,328 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 108: setrep: existent directory (absolute path)
2008-04-30 09:45:55,328 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 109: setrep: existent directory (relative path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 110: setrep: non existent file (absolute path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 111: setrep: non existent file (relative path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 112: touchz: touching file (absolute path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 113: touchz: touching file(relative path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 114: touchz: touching many files
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 115: touchz: touching already existing file
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 116: test: non existent file (absolute path)
2008-04-30 09:45:55,329 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 117: test: non existent file (relative path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 118: test: non existent directory (absolute path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 119: test: non existent directory (relative path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 120: stat: statistics about file(absolute path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 121: stat: statistics about file(relative path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 122: stat: statistics about directory(absolute path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 123: stat: statistics about directory(relative path)
2008-04-30 09:45:55,330 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 124: stat: statistics about files (absolute path) using globbing
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 125: stat: statistics about files (relative path) using globbing
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 126: stat: statistics about file or directory(absolute path) that does not exist
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 127: stat: statistics about file or directory(relative path) that does not exist
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 128: tail: contents of file(absolute path)
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 129: tail: contents of file(relative path)
2008-04-30 09:45:55,331 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 130: tail: contents of files(absolute path) using globbing
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 131: tail: contents of files(relative path) using globbing
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 132: tail: contents of file(absolute path) that does not exist
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 133: tail: contents of file(relative path) that does not exist
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 134: tail: contents of directory(absolute path)
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 135: tail: contents of directory(relative path)
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 136: moveFromLocal: moving non existent file(absolute path)
2008-04-30 09:45:55,332 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 137: moveFromLocal: moving non existent file(relative path)
2008-04-30 09:45:55,333 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 138: moveFromLocal: moving many files into an existing file
2008-04-30 09:45:55,333 INFO cli.TestCLI (TestCLI.java:displayResults(271)) - 139: moveFromLocal: moving many files into a non existent directory

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Attachment: H3100.patch

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12593415#action_12593415 ]

Hadoop QA commented on HADOOP-3100:
-----------------------------------

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12381203/H3100.patch
against trunk revision 645773.

@author +1. The patch does not contain any @author tags.

tests included +1. The patch appears to include 327 new or modified tests.

patch -1. The patch command could not apply the patch.

Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2353/console

This message is automatically generated.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Nigel Daley updated HADOOP-3100:
--------------------------------

Attachment: H3100.patch

Update patch for trunk

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12594067#action_12594067 ]

Hadoop QA commented on HADOOP-3100:
-----------------------------------

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12381373/H3100.patch
against trunk revision .

+1 @author. The patch does not contain any @author tags.

+1 tests included. The patch appears to include 327 new or modified tests.

+1 javadoc. The javadoc tool did not generate any warning messages.

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

+1 findbugs. The patch does not introduce any new Findbugs warnings.

-1 release audit. The applied patch generated 213 release audit warnings (more than the trunk's current 207 warnings).

-1 core tests. The patch failed core unit tests.

+1 contrib tests. The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2389/testReport/
Release audit warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2389/artifact/trunk/current/releaseAuditDiffWarnings.txt
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2389/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2389/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2389/console

This message is automatically generated.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Status: Open (was: Patch Available)

Canceling patch as it needs an update for trunk due to command like output changes in trunk

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Attachment: H3100.patch

Updated patch to work with changes in 0.18.0

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Status: Patch Available (was: Open)

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12605782#action_12605782 ]

Hadoop QA commented on HADOOP-3100:
-----------------------------------

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12384159/H3100.patch
against trunk revision 668832.

+1 @author. The patch does not contain any @author tags.

+1 tests included. The patch appears to include 327 new or modified tests.

+1 javadoc. The javadoc tool did not generate any warning messages.

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

+1 findbugs. The patch does not introduce any new Findbugs warnings.

-1 release audit. The applied patch generated 209 release audit warnings (more than the trunk's current 203 warnings).

-1 core tests. The patch failed core unit tests.

+1 contrib tests. The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2674/testReport/
Release audit warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2674/artifact/trunk/current/releaseAuditDiffWarnings.txt
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2674/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2674/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2674/console

This message is automatically generated.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606240#action_12606240 ]

Nigel Daley commented on HADOOP-3100:
-------------------------------------

Release audit warning are ok. The new files don't need Apache headers.

Test failure is due to org.apache.hadoop.cli.util.TestData not having any test methods. This class should be renamed so that it doesn't start with the work "Test".

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Owen O'Malley updated HADOOP-3100:
----------------------------------

Status: Open (was: Patch Available)

canceling patch until Nigel's suggestion of changing the class name is done.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Attachment: H3100.patch

Updated patch: renamed TestData to CLITestData

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Status: Patch Available (was: Open)

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12607782#action_12607782 ]

Hadoop QA commented on HADOOP-3100:
-----------------------------------

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12384623/H3100.patch
against trunk revision 671277.

+1 @author. The patch does not contain any @author tags.

+1 tests included. The patch appears to include 327 new or modified tests.

+1 javadoc. The javadoc tool did not generate any warning messages.

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

+1 findbugs. The patch does not introduce any new Findbugs warnings.

-1 release audit. The applied patch generated 213 release audit warnings (more than the trunk's current 207 warnings).

+1 core tests. The patch passed core unit tests.

+1 contrib tests. The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2727/testReport/
Release audit warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2727/artifact/trunk/current/releaseAuditDiffWarnings.txt
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2727/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2727/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2727/console

This message is automatically generated.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12608838#action_12608838 ]

Mukund Madhugiri commented on HADOOP-3100:
------------------------------------------

Release audit warning are ok. The new files don't need Apache headers.

The patch is good to go

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mukund Madhugiri updated HADOOP-3100:
-------------------------------------

Resolution: Fixed
Fix Version/s: 0.18.0
Status: Resolved (was: Patch Available)

I just committed this.

[ https://issues.apache.org/jira/browse/HADOOP-3100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609920#action_12609920 ]

Hudson commented on HADOOP-3100:
--------------------------------

Integrated in Hadoop-trunk #536 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/536/])