Issue Details (XML | Word | Printable)

Key: HADOOP-2563
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Lohit Vijayarenu
Reporter: Doug Cutting
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

Remove deprecated FileSystem#listPaths()

Created: 09/Jan/08 06:49 PM   Updated: 21/May/08 08:05 PM
Return to search
Component/s: fs
Affects Version/s: None
Fix Version/s: 0.17.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works HADOOP-2563-1.patch 2008-03-28 10:31 PM Lohit Vijayarenu 32 kB
Text File Licensed for inclusion in ASF works HADOOP-2563-2.patch 2008-03-29 12:02 AM Lohit Vijayarenu 32 kB
Text File Licensed for inclusion in ASF works HADOOP-2563-3.patch 2008-03-29 04:39 AM Lohit Vijayarenu 32 kB
Issue Links:
Dependants
 

Hadoop Flags: Incompatible change
Release Note: Removes deprecated method FileSystem#listPaths()
Resolution Date: 29/Mar/08 10:18 PM


 Description  « Hide
FileSystem#listPaths() has been deprecated for a few releases, and we should now remove it, upgrading everything to use FileSystem#listStatus().

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Robert Chansler added a comment - 28/Mar/08 12:07 AM
If deprecated in 0.15, then remove for 0.17.
Otherwise set the fix version for this issue to 0.18.

Lohit Vijayarenu added a comment - 28/Mar/08 10:31 PM
Attaching a patch which removes all deprecated listPaths()

Hadoop QA added a comment - 28/Mar/08 11:39 PM
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378824/HADOOP-2563-1.patch
against trunk revision 619744.

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

tests included +1. The patch appears to include 47 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/2093/console

This message is automatically generated.


Lohit Vijayarenu added a comment - 29/Mar/08 12:02 AM
By mistake uploaded wrong patch which failed to apply. I tested this locally and it applies on trunk. Retrying.

Hadoop QA added a comment - 29/Mar/08 01:58 AM
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378830/HADOOP-2563-2.patch
against trunk revision 619744.

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

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

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

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

release audit +1. The applied patch does not generate any new release audit warnings.

findbugs -1. The patch appears to introduce 1 new Findbugs warnings.

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

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

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

This message is automatically generated.


Hadoop QA added a comment - 29/Mar/08 05:52 AM
+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378839/HADOOP-2563-3.patch
against trunk revision 619744.

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

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

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

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

release audit +1. The applied patch does not generate any new release audit warnings.

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

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

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

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

This message is automatically generated.


Chris Douglas added a comment - 29/Mar/08 10:18 PM
I just committed this. Thanks Lohit!

Hudson added a comment - 30/Mar/08 12:05 PM

Robert Chansler added a comment - 14/Apr/08 04:30 PM
Noted as incompatible in changes.txt