Issue Details (XML | Word | Printable)

Key: HADOOP-2928
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Lohit Vijayarenu
Reporter: Tsz Wo (Nicholas), SZE
Votes: 0
Watchers: 0
Operations

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

Remove deprecated methods getContentLength() in ClientProtocol, NameNode, FileSystem, DistributedFileSystem and DFSClient

Created: 03/Mar/08 06:31 PM   Updated: 08/Jul/09 04:42 PM
Component/s: None
Affects Version/s: None
Fix Version/s: 0.18.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works HADOOP-2928.patch 2008-04-15 01:39 AM Lohit Vijayarenu 9 kB

Hadoop Flags: Reviewed
Resolution Date: 15/Apr/08 05:59 PM


 Description  « Hide
The methods getContentLength() in ClientProtocol, NameNode, FileSystem, DistributedFileSystem and DFSClient are deprecated HADOOP-2219

 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:05 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 09:24 PM
This was deprecated in 0.17. So, making the fix version for 0.18

Lohit Vijayarenu added a comment - 15/Apr/08 01:39 AM
Attached patch removes getContentLength()

Hadoop QA added a comment - 15/Apr/08 07:08 AM
+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12380130/HADOOP-2928.patch
against trunk revision 645773.

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

tests included +1. The patch appears to include 6 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/2234/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2234/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2234/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2234/console

This message is automatically generated.


Tsz Wo (Nicholas), SZE added a comment - 15/Apr/08 05:19 PM
+1 patch looks good

Raghu Angadi added a comment - 15/Apr/08 05:59 PM
I just committed this. Thanks Lohit.

Hudson added a comment - 16/Apr/08 12:38 PM